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. 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? johannes -- 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
