Re: [PATCH] mac80211: Remove dummy STA on deauth

[Johannes Berg <johannes@sipsolutions.net>]

On Tue, 2012-01-17 at 11:19 +0100, Pontus Fuchs wrote:
> Under certain conditions a dummy STA entry could be left in the
> station list.

This happens when a deauth is requested while associating.

>  This causes subsequent connection attempts to fail.
> 
> Logs look like this:
> 
> authenticate with 00:23:69:aa:dd:7b (try 1)
> authenticated
> failed to insert Dummy STA entry for the AP (error -17)
> deauthenticating from 00:23:69:aa:dd:7b by local choice (reason=2)
> 
> Fix this by making sure to remove the dummy STA on deauth.
> 
> Signed-off-by: Pontus Fuchs <pontus.fuchs@gmail.com>
> ---
> 
> The problem was introduced by the following commit added in v3.1:
> mac80211: fix race condition between assoc_done and first EAP packet
> 
> This problem is fixed in the recent patch series from 
> Johannes Berg: [PATCH 0/8] more station handling refactoring
> 
> If you need a smaller fix this might fit you.

It doesn't fit me :-)
Could you please make a fix that also fixes the missing
finish_tx_sync()?

Also, I think the code should be moved to the right place inside the
work cancelling in mgd_deauth().

johannes