cvs commit: src/sys/net if_vlan.c
yar at FreeBSD.org
Thu Aug 3 09:59:10 UTC 2006
yar 2006-08-03 09:59:09 UTC
FreeBSD src repository
Should vlan_input() ever be called with ifp pointing to a non-Ethernet
interface, do not just assign -1 to tag because it breaks the logic of
the code to follow. The better way is to handle this case as an unsupported
protocol and return unless INVARIANTS is in effect and we can panic.
Panic is good there because the scenario can happen only because of a
coding error elsewhere.
We also should show the interface name in the panic message for easier
debugging of the problem, should it ever emerge.
Submitted by: qingli (initially)
Revision Changes Path
1.109 +5 -4 src/sys/net/if_vlan.c
More information about the cvs-src