cvs commit: src/sys/net if_vlan.c
Sam Leffler
sam at errno.com
Thu Aug 3 17:11:14 UTC 2006
Yar Tikhiy wrote:
> yar 2006-08-03 09:59:09 UTC
>
> FreeBSD src repository
>
> Modified files:
> sys/net if_vlan.c
> Log:
> 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.
Introducing a panic in a place where you can trivially recover is bad
regardless of why you got there. Many people run production systems
with INVARIANTS turned on. Is it now possible to send a "packet of
death" by exploiting this code path?
Sam
More information about the cvs-src
mailing list