diff options
author | rrs <rrs@FreeBSD.org> | 2007-05-08 14:32:53 +0000 |
---|---|---|
committer | rrs <rrs@FreeBSD.org> | 2007-05-08 14:32:53 +0000 |
commit | 494599278742a1165b67db90d09111a09b94fdde (patch) | |
tree | 118f90542ed683b5fc2b1e91576d2fc62788dfd2 /sys/netinet/sctp_var.h | |
parent | 99bfdc548a5093b3e2a573df1f917f1671f05f11 (diff) | |
download | FreeBSD-src-494599278742a1165b67db90d09111a09b94fdde.zip FreeBSD-src-494599278742a1165b67db90d09111a09b94fdde.tar.gz |
- Static analyisis fixes for cisco's commit (this is equivilant
to the coverity tool.. may even be the same one.. not sure).
- A bug in the way sctp_abort() and friends were
setting the IP_CLOSE flag.. and NOT passing the
last argument as a (,1)... so that things would
get freed..
Diffstat (limited to 'sys/netinet/sctp_var.h')
-rw-r--r-- | sys/netinet/sctp_var.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/netinet/sctp_var.h b/sys/netinet/sctp_var.h index ae68b23..a7a3f6b 100644 --- a/sys/netinet/sctp_var.h +++ b/sys/netinet/sctp_var.h @@ -121,9 +121,9 @@ extern struct pr_usrreqs sctp_usrreqs; #define sctp_free_remote_addr(__net) { \ if ((__net)) { \ if (atomic_fetchadd_int(&(__net)->ref_count, -1) == 1) { \ - SCTP_OS_TIMER_STOP(&(__net)->rxt_timer.timer); \ - SCTP_OS_TIMER_STOP(&(__net)->pmtu_timer.timer); \ - SCTP_OS_TIMER_STOP(&(__net)->fr_timer.timer); \ + (void)SCTP_OS_TIMER_STOP(&(__net)->rxt_timer.timer); \ + (void)SCTP_OS_TIMER_STOP(&(__net)->pmtu_timer.timer); \ + (void)SCTP_OS_TIMER_STOP(&(__net)->fr_timer.timer); \ if ((__net)->ro.ro_rt) { \ RTFREE((__net)->ro.ro_rt); \ (__net)->ro.ro_rt = NULL; \ |