Date: Thu, 22 Dec 2005 05:05:00 +0000 (UTC) From: Ed Maste <emaste@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/net if.c Message-ID: <200512220505.jBM550VI078230@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
emaste 2005-12-22 05:05:00 UTC
FreeBSD src repository
Modified files: (Branch: RELENG_5)
sys/net if.c
Log:
MFC r1.235, r1.237, r1.238:
(1.235)
- Introduce a helper function if_setflag() containing the code common
to ifpromisc() and if_allmulti() instead of duplicating the code poorly,
with different bugs.
- Call ifp->if_ioctl() in a consistent way: always use more compatible C
syntax and check whether ifp->if_ioctl is not NULL prior to the call.
(1.237)
Rename equal() macro to sa_equal(), which matches the definitions
of sa_equal() in other files, and makes it more clear what equal()
is comparing.
(1.238)
In multicast routines:
Compare pointers with NULL rather than treating them as booleans.
Compare pointers with NULL rather than 0 to make it more clear
they are pointers.
Assign pointers value of NULL rather than 0 to make it more clear
they are pointers.
Approved by: scottl
Silence from: rwatson (mentor)
Revision Changes Path
1.199.2.19 +127 -100 src/sys/net/if.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200512220505.jBM550VI078230>