diff options
author | rrs <rrs@FreeBSD.org> | 2015-11-13 22:51:35 +0000 |
---|---|---|
committer | rrs <rrs@FreeBSD.org> | 2015-11-13 22:51:35 +0000 |
commit | dc494194a2039a69fc3073f4ef4d628b955a0b25 (patch) | |
tree | 8d27e0d0e4a2ccb1fcf414f3765ec6a6e6de9611 /sys/net/if_llatbl.c | |
parent | e3347af437af55376a333371d3c62fb46c99b67d (diff) | |
download | FreeBSD-src-dc494194a2039a69fc3073f4ef4d628b955a0b25.zip FreeBSD-src-dc494194a2039a69fc3073f4ef4d628b955a0b25.tar.gz |
This fixes several places where callout_stops return is examined. The
new return codes of -1 were mistakenly being considered "true". Callout_stop
now returns -1 to indicate the callout had either already completed or
was not running and 0 to indicate it could not be stopped. Also update
the manual page to make it more consistent no non-zero in the callout_stop
or callout_reset descriptions.
MFC after: 1 Month with associated callout change.
Diffstat (limited to 'sys/net/if_llatbl.c')
-rw-r--r-- | sys/net/if_llatbl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/net/if_llatbl.c b/sys/net/if_llatbl.c index 1c16b2d..0a89225 100644 --- a/sys/net/if_llatbl.c +++ b/sys/net/if_llatbl.c @@ -394,7 +394,7 @@ lltable_free(struct lltable *llt) IF_AFDATA_WUNLOCK(llt->llt_ifp); LIST_FOREACH_SAFE(lle, &dchain, lle_chain, next) { - if (callout_stop(&lle->lle_timer)) + if (callout_stop(&lle->lle_timer) > 0) LLE_REMREF(lle); llentry_free(lle); } |