ndis lor: hal preemption lock
Paul B. Mahol
onemda at gmail.com
Wed Jul 15 07:59:25 UTC 2009
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.
--
Paul
More information about the freebsd-net
mailing list