[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Sun, 04 May 2008 17:59:43 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: xemul@...nvz.org
Cc: johannes@...solutions.net, linville@...driver.com,
linux-wireless@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH][MAC80211]: Do not free net device after it is
unregistered.
From: Pavel Emelyanov <xemul@...nvz.org>
Date: Sun, 04 May 2008 10:53:45 +0400
> The error path in ieee80211_register_hw() may call the unregister_netdev()
> and right after it - the free_netdev(), which is wrong, since the
> unregister releases the device itself.
>
> So the proposed fix is to NULL the local->mdev after unregister is done
> and check this before calling free_netdev().
>
> I checked - no code uses the local->mdev after unregister in this error
> path (but even if some did this would be a BUG).
>
> Signed-off-by: Pavel Emelyanov <xemul@...nvz.org>
I'll take this.
Applied, thanks Pavel.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists