diff options
author | rwatson <rwatson@FreeBSD.org> | 2007-05-27 17:38:36 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2007-05-27 17:38:36 +0000 |
commit | 5b4b005b06b0017c69716ee1d55976682104268e (patch) | |
tree | 8f85a9373617dcc5b76d40c3040242053e83bddf /sys/kern/subr_mbpool.c | |
parent | 60d85d52f56734ab2c0d949030629b6e7820c5e8 (diff) | |
download | FreeBSD-src-5b4b005b06b0017c69716ee1d55976682104268e.zip FreeBSD-src-5b4b005b06b0017c69716ee1d55976682104268e.tar.gz |
Add parens around *free in *free++ in mbp_count() so that mbp_count()
actually works. mbp_count() turns out only to be used in debugging code
in if_patm_intr.c, so this bug did not affect much in practice.
Found with: Coverity Prevent(tm)
CID: 1943
Diffstat (limited to 'sys/kern/subr_mbpool.c')
-rw-r--r-- | sys/kern/subr_mbpool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/subr_mbpool.c b/sys/kern/subr_mbpool.c index 400c6da..ab80a5e 100644 --- a/sys/kern/subr_mbpool.c +++ b/sys/kern/subr_mbpool.c @@ -338,7 +338,7 @@ mbp_count(struct mbpool *p, u_int *used, u_int *card, u_int *free) } mtx_lock(&p->free_lock); SLIST_FOREACH(cf, &p->free_list, link) - *free++; + (*free)++; mtx_unlock(&p->free_lock); } |