On Sun, Nov 27, 2011 at 11:38, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Sun, 2011-11-27 at 02:23 -0500, Nikolay Martynov wrote: >> Currently if addBA respones comes in just after addba_resp_timer has >> expired we still accept addBA response and (try to) open agg >> session. This patch fixes this race condition and makes sure that if >> addba_resp_timer has expired addBA response is not longer accepted and >> we do not try to open half-closed session. > I just sent a patch to Norbert that looks a bit the same. I prefer yours though, as Johannes said in the other thread, it checks the stop condition better. I also saw a few other holes in the state machine. Basically we don't clear flags: i.e. I didn't see where we clear the driver ready flag. This is the kind of things that can lead to weird stuff. -- 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
