On Mon, 2011-03-28 at 12:57 +0200, ext Johannes Berg wrote: > On Mon, 2011-03-28 at 10:11 +0300, juuso.oikarinen@xxxxxxxxx wrote: > > From: Juuso Oikarinen <juuso.oikarinen@xxxxxxxxx> > > > > This patch adds to the fix "fix BSS double-unlinking" > > (commit 3207390a8b58bfc1335750f91cf6783c48ca19ca) by Johannes Berg. > > > > It turns out, that the double-unlinking scenario can also occur if expired > > BSS elements are removed whilst an interface is performing association. > > Ack, > > > To work around that, replace list_del with list_del_init also in the > > "cfg80211_bss_expire" function, so that the check for whether the BSS still is > > in the list works correctly in cfg80211_bss_expire. > > but I think you mean cfg80211_unlink_bss there at the end. Yeah, right. > Might be worthwhile to create a new static function that rolls up this > code (list_del_init, rb_erase, kref_put) between the two functions? Sure, can do. -Juuso -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
