From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Date: Mon, 14 Mar 2016 08:15:33 +0200 Subject: [v2, resend] b43: Fix memory leaks in b43_bus_dev_ssb_init and b43_bus_dev_bcma_init In-Reply-To: <20160311173047.785c2b24@wiggum> ("Michael =?utf-8?Q?B=C3=BCs?= =?utf-8?Q?ch=22's?= message of "Fri, 11 Mar 2016 17:30:47 +0100") References: <1452949690-16387-1-git-send-email-baijiaju1990@163.com> <20160311162230.GA5436@sudip-laptop> <20160311173047.785c2b24@wiggum> Message-ID: <87egbdmvp6.fsf@kamboji.qca.qualcomm.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Michael =?utf-8?Q?B=C3=BCsch?= Cc: Sudip Mukherjee , Jia-Ju Bai , linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org, joe@perches.com Michael B?sch writes: > On Fri, 11 Mar 2016 21:52:30 +0530 > Sudip Mukherjee wrote: > >> On Sat, Jan 16, 2016 at 09:08:10PM +0800, Jia-Ju Bai wrote: >> > The memory allocated by kzalloc in b43_bus_dev_ssb_init and >> > b43_bus_dev_bcma_init is not freed. >> > This patch fixes the bug by adding kfree in b43_ssb_remove, >> > b43_bcma_remove and error handling code of b43_bcma_probe. >> > >> > Thanks Michael for his suggestion. >> > >> > Signed-off-by: Jia-Ju Bai >> >> The patch did not apply cleanly. I had to edit the patch to point to >> drivers/net/wireless/broadcom/b43/main.c >> >> For CONFIG_B43_SSB part- >> Tested-by: Sudip Mukherjee > > > Ok thanks a lot. > We possibly don't get an bcma tester anytime soon. As this is quite an > important fix, we should probably apply it. > > Sudip, can you please update the patch so it applies cleanly on the > current tree? You can add my acked-by if you want. > > Kalle, can you please apply it afterwards? Will do. -- Kalle Valo