Sure enough, your patch works well. The end effect is correct, in that despite the reception of the DEAUTH, we are successfully able to process a successful auth response on the second try. Do I take it that this change will make it upstream since this is the case? Have a happy wedding, and no git merge until after the ceremony! :-) -- Paul On Fri, Jul 2, 2010 at 11:13 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2010-07-02 at 11:09 -0700, Paul Stewart wrote: > Can you try the patch below instead of yours? I'll explain it a bit more > later, but my church wedding ceremony is tomorrow :) > > johannes > > --- wireless-testing.orig/net/wireless/mlme.c 2010-07-02 20:12:19.000000000 +0200 > +++ wireless-testing/net/wireless/mlme.c 2010-07-02 20:12:43.000000000 +0200 > @@ -44,10 +44,10 @@ void cfg80211_send_rx_auth(struct net_de > } > } > > - WARN_ON(!done); > - > - nl80211_send_rx_auth(rdev, dev, buf, len, GFP_KERNEL); > - cfg80211_sme_rx_auth(dev, buf, len); > + if (done) { > + nl80211_send_rx_auth(rdev, dev, buf, len, GFP_KERNEL); > + cfg80211_sme_rx_auth(dev, buf, len); > + } > > wdev_unlock(wdev); > } > > > -- 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