Return-path: Received: from mx1.redhat.com ([209.132.183.28]:38840 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756175AbcCUNaH (ORCPT ); Mon, 21 Mar 2016 09:30:07 -0400 Date: Mon, 21 Mar 2016 14:28:14 +0100 From: Stanislaw Gruszka To: Julian Calaby Cc: Kalle Valo , Johannes Berg , Sara Sharon , Emmanuel Grumbach , linux-wireless@vger.kernel.org, Jia-Ju Bai Subject: Re: [PATCH MOREWORK 18/19] iwl4965: Fix more memory leaks in __il4965_up() Message-ID: <20160321132813.GA31456@redhat.com> (sfid-20160321_143218_081396_95A022E7) References: <6bec5a9b55f13d40c2803f0fa60173451d5ea14a.1458262312.git.julian.calaby@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <6bec5a9b55f13d40c2803f0fa60173451d5ea14a.1458262312.git.julian.calaby@gmail.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Fri, Mar 18, 2016 at 01:29:11PM +1100, Julian Calaby wrote: > In some of the non-success return paths, the memory allocated by > iwl4965_sta_alloc_lq() in iwl4965_alloc_bcast_station() is not freed. > > In particular: > - if the card isn't ready after il4965_prepare_card_hw() > - if the card is hardware-rfkilled > > In the hardware rfkilled path, the driver enables the rfkill > interrupt. When the card is unrfkilled and this interrupt is raised > we end up calling il4965_bg_restart() which calls __il4965_up() which > calls iwl4965_alloc_bcast_station() again. > > Suggested-by: Jia-Ju Bai > Signed-off-by: Julian Calaby > > --- > > This is only compile tested as I don't have compatible hardware. I also > don't know the driver enough to know that this is truly correct - however > it looks right and I stand by my analysis. > > Could someone else please review this? Looks ok. Acked-by: Stanislaw Gruszka