On Tue, 2014-06-24 at 13:55 +0200, Michal Kazior wrote: > >> + if (WARN_ON(!ctx->replace_ctx)) > >> + return -EINVAL; > > > > This also leaks "vif_chsw". > > This should be a `goto` too. This was the first thing I'd actually looked at, and there's currently no label that frees vif_chsw :) > I'll re-spin once I fix all the problems that I'm aware of now. Thanks. 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