On 27/07/12 13:42, Johannes Berg wrote:
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?
Yes, that'll work too.
-- Pozdrawiam / Best regards, Michal Kazior.
--
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