Date: Sat, 31 Jul 2010 21:33:18 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/netinet if_ether.c src/sys/netinet6 nd6.c Message-ID: <201007312133.o6VLXftc045442@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
bz 2010-07-31 21:33:18 UTC
FreeBSD src repository
Modified files:
sys/netinet if_ether.c
sys/netinet6 nd6.c
Log:
SVN rev 210703 on 2010-07-31 21:33:18Z by bz
Document the mandatory argument to the arptimer() and
nd6_llinfo_timer() functions with a KASSERT().
Note: there is no need to return after panic.
In the legacy IP case, only assign the arg after the check,
in the IPv6 case, remove the extra checks for the table and
interface as they have to be there unless we freed and forgot
to cancel the timer. It doesn't matter anyway as we would
panic on the NULL pointer deref immediately and the bug is
elsewhere.
This unifies the code of both address families to some extend.
Reviewed by: rwatson
MFC after: 6 days
Revision Changes Path
1.220 +3 -5 src/sys/netinet/if_ether.c
1.135 +2 -7 src/sys/netinet6/nd6.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201007312133.o6VLXftc045442>