diff options
author | blueswir1 <blueswir1@c046a42c-6fe2-441c-8c8c-71466251a162> | 2009-03-07 15:32:56 +0000 |
---|---|---|
committer | blueswir1 <blueswir1@c046a42c-6fe2-441c-8c8c-71466251a162> | 2009-03-07 15:32:56 +0000 |
commit | 511d2b140f3ff2f80d14637cdc2f29743a2daa51 (patch) | |
tree | 6cf21d1c95846f1c86879c60dffbe6a3eaa8aa6d /slirp/mbuf.c | |
parent | c2764719914ff0c4d6c06adafea17629600f21ba (diff) |
Sparse fixes: NULL use, header order, ANSI prototypes, static
Fix Sparse warnings:
* use NULL instead of plain 0
* rearrange header include order to avoid redefining types accidentally
* ANSIfy SLIRP
* avoid "restrict" keyword
* add static
git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@6736 c046a42c-6fe2-441c-8c8c-71466251a162
Diffstat (limited to 'slirp/mbuf.c')
-rw-r--r-- | slirp/mbuf.c | 29 |
1 files changed, 10 insertions, 19 deletions
diff --git a/slirp/mbuf.c b/slirp/mbuf.c index 655de41805..1d30a630c6 100644 --- a/slirp/mbuf.c +++ b/slirp/mbuf.c @@ -29,7 +29,7 @@ int mbuf_max = 0; #define SLIRP_MSIZE (IF_MTU + IF_MAXLINKHDR + sizeof(struct m_hdr ) + 6) void -m_init() +m_init(void) { m_freelist.m_next = m_freelist.m_prev = &m_freelist; m_usedlist.m_next = m_usedlist.m_prev = &m_usedlist; @@ -44,7 +44,7 @@ m_init() * which tells m_free to actually free() it */ struct mbuf * -m_get() +m_get(void) { register struct mbuf *m; int flags = 0; @@ -72,16 +72,15 @@ m_get() m->m_size = SLIRP_MSIZE - sizeof(struct m_hdr); m->m_data = m->m_dat; m->m_len = 0; - m->m_nextpkt = 0; - m->m_prevpkt = 0; + m->m_nextpkt = NULL; + m->m_prevpkt = NULL; end_error: DEBUG_ARG("m = %lx", (long )m); return m; } void -m_free(m) - struct mbuf *m; +m_free(struct mbuf *m) { DEBUG_CALL("m_free"); @@ -115,8 +114,7 @@ m_free(m) * an M_EXT data segment */ void -m_cat(m, n) - register struct mbuf *m, *n; +m_cat(struct mbuf *m, struct mbuf *n) { /* * If there's no room, realloc @@ -133,9 +131,7 @@ m_cat(m, n) /* make m size bytes large */ void -m_inc(m, size) - struct mbuf *m; - int size; +m_inc(struct mbuf *m, int size) { int datasize; @@ -170,9 +166,7 @@ m_inc(m, size) void -m_adj(m, len) - struct mbuf *m; - int len; +m_adj(struct mbuf *m, int len) { if (m == NULL) return; @@ -192,9 +186,7 @@ m_adj(m, len) * Copy len bytes from m, starting off bytes into n */ int -m_copy(n, m, off, len) - struct mbuf *n, *m; - int off, len; +m_copy(struct mbuf *n, struct mbuf *m, int off, int len) { if (len > M_FREEROOM(n)) return -1; @@ -211,8 +203,7 @@ m_copy(n, m, off, len) * Fortunately, it's not used often */ struct mbuf * -dtom(dat) - void *dat; +dtom(void *dat) { struct mbuf *m; |