From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761991AbcIXK2G (ORCPT ); Sat, 24 Sep 2016 06:28:06 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:54216 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754016AbcIXK2D (ORCPT ); Sat, 24 Sep 2016 06:28:03 -0400 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 87583612D0 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=kvalo@codeaurora.org From: Kalle Valo To: =?utf-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Cc: =?utf-8?Q?Rafa=C5=82_Mi=C5=82ecki?= , Arend van Spriel , Franky Lin , Hante Meuleman , Pieter-Paul Giesberts , "Franky \(Zhenhui\) Lin" , linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] brcmfmac: fix memory leak in brcmf_fill_bss_param In-Reply-To: <20160921062327.28729-1-zajec5@gmail.com> (=?utf-8?Q?=22Rafa?= =?utf-8?Q?=C5=82_Mi=C5=82ecki=22's?= message of "Wed, 21 Sep 2016 08:23:24 +0200") References: <20160921062327.28729-1-zajec5@gmail.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) Date: Sat, 24 Sep 2016 13:27:57 +0300 Message-ID: <87shsp1tjm.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id u8OASCf4028734 Rafał Miłecki writes: > From: Rafał Miłecki > > This function is called from get_station callback which means that every > time user space was getting/dumping station(s) we were leaking 2 KiB. > > Signed-off-by: Rafał Miłecki > Fixes: 1f0dc59a6de ("brcmfmac: rework .get_station() callback") > Cc: stable@vger.kernel.org # 4.2+ > --- > Kalle, ideally this should go as 4.8 fix, but I'm aware it's quite late. > If you are not planning to send another pull request, just get it for > the next release and let's let stable guys backport it. An old memory leak is not severe enough for 4.8 at this stage, so I'll queue this to 4.9. BTW, either my Gnus or my SMTP server (I haven't bothered to check yet why exactly) don't like the names with style of "(open list:NETWORKING DRIVERS)" in the CC list, I have to edit them away everytime I reply. Does anyone have any ideas why that's happening just to me? -- Kalle Valo