CVSROOT:        /cvs
Module name:    src
Changes by:     [email protected]    2020/07/29 06:13:28

Modified files:
        sbin/ifconfig  : brconfig.c 

Log message:
Remove redundant bridge checks

bridge_status() and switch_status() do the regular sanity check with
SIOCGIFFLAGS, but both functions also call is_switch() and bridge_status()
also calls is_bridge().

is_bridge() checks SIOCGIFFLAGS again, then both is_*() helpers finally do
driver specific ioctl(2) calls to test whether the given interface is
indeed a bridge(4) or a switch(4).

SIOCGIFFLAGS serves no purpose here and is taken care of in ifconfig.c's
getinfo(), so remove its calls from brconfig.c entirely.

OK dlg

Reply via email to