On Fri, 2012-07-27 at 13:42 +0200, Michal Kazior wrote: > On 27/07/12 13:16, Johannes Berg wrote: > > --- a/net/mac80211/chan.c > > +++ b/net/mac80211/chan.c > > @@ -237,7 +237,10 @@ ieee80211_new_chanctx(struct ieee80211_local *local, > > > > list_add(&ctx->list, &local->chanctx_list); > > > > - drv_add_chanctx(local, ctx); > > + if (drv_add_chanctx(local, ctx)) { > > + kfree(ctx); > > We should also do a list_del() here. Good catch. Or just move the list_add() later? 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