ndis lor: hal preemption lock

Paul B. Mahol onemda at gmail.com
Sun Jul 19 17:28:13 UTC 2009


On 7/15/09, Paul B. Mahol <onemda at gmail.com> wrote:
> On 7/4/09, Paul B. Mahol <onemda at gmail.com> wrote:
>> On 6/18/09, Coleman Kane <cokane at freebsd.org> wrote:
>>> I've committed this one as r194432.
>>
>> Ah, that one introduced regression.
>> Switching ndisX up before creating vap will cause panic.
>> Here is fix:
>>
>> --- /sys/dev/if_ndis/if_ndis.c  2009-06-28 09:15:54.000000000 +0000
>> +++ if_ndis.c   2009-07-04 10:23:41.000000000 +0000
>> @@ -2292,6 +2292,8 @@
>>         ifp = sc->ifp;
>>         ic = ifp->if_l2com;
>>         vap = TAILQ_FIRST(&ic->ic_vaps);
>> +       if (vap == NULL)
>> +               return;
>>
>>         if (!NDIS_INITIALIZED(sc)) {
>>                 DPRINTF(("%s: NDIS not initialized\n", __func__));
>
> Bump!
>
> Please commit.

Beep!

kern/136895

-- 
Paul


More information about the freebsd-current mailing list