On Sun, 23 Sep 2001, Jonathan Lemon wrote:
> > Right; rather than failing the detach routines will fuss about it so you
> > know exactly how you screwed up.  I don't see this papering anything over.
> 
> Because this is not a normal operational error.  If anything, the
> statement should be a KASSERT(), but I don't really see the need for it.

bpfdetach() already has a test for this case; adding one to
if_detach() doesn't seem like a bad thing so long as abusing if_index
isn't a problem.

-- 
| Matthew N. Dodd  | '78 Datsun 280Z | '75 Volvo 164E | FreeBSD/NetBSD  |
| [EMAIL PROTECTED] |       2 x '84 Volvo 245DL        | ix86,sparc,pmax |
| http://www.jurai.net/~winter |  For Great Justice!  | ISO8802.5 4ever |


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-net" in the body of the message

Reply via email to