* Alfred Perlstein <[EMAIL PROTECTED]> [030316 21:19] wrote:
> um..
>
...
840 _FLAGS_OUTRANGE) {
841 WI_UNLOCK(sc);
842 return;
843 }
844 KASSERT((ifp->if_flags & IFF_OACTIVE) == 0,
845 ("wi_start: if_flags %x\n", ifp->if_flags));
846
847 memset(&frmhdr, 0, sizeof(frmhdr));
>
> What's up here?
It's a race, we shouldn't be inspecting the ifp without a lock.
I think this kassert should be removed.
Do you guys concurr?
--
-Alfred Perlstein [EMAIL PROTECTED]
'Instead of asking why a piece of software is using "1970s technology,"
start asking why software is ignoring 30 years of accumulated wisdom.'
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
