cvs commit: src/sys/net if_vlan.c

[Sam Leffler]

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