On Fri, 2012-07-27 at 13:42 +0200, Michal Kazior wrote:

On 27/07/12 13:16, Johannes Berg wrote:

quoted

--- 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