Return-path: Received: from mail-wr0-f174.google.com ([209.85.128.174]:35609 "EHLO mail-wr0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751090AbdFTTxS (ORCPT ); Tue, 20 Jun 2017 15:53:18 -0400 Received: by mail-wr0-f174.google.com with SMTP id y25so65936649wrd.2 for ; Tue, 20 Jun 2017 12:53:17 -0700 (PDT) Subject: Re: [PATCH] cfg80211: Fix a memory leak in error handling path in 'brcmf_cfg80211_attach' To: Christophe JAILLET , franky.lin@broadcom.com, hante.meuleman@broadcom.com, kvalo@codeaurora.org, rafal@milecki.pl, pieter-paul.giesberts@broadcom.com Cc: linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org References: <20170620062205.16078-1-christophe.jaillet@wanadoo.fr> From: Arend van Spriel Message-ID: <2343c662-f70d-1e76-d863-5f6bb66c269b@broadcom.com> (sfid-20170620_215339_680418_E349B1A0) Date: Tue, 20 Jun 2017 21:53:15 +0200 MIME-Version: 1.0 In-Reply-To: <20170620062205.16078-1-christophe.jaillet@wanadoo.fr> Content-Type: text/plain; charset=utf-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 20-06-17 08:22, Christophe JAILLET wrote: > If 'wiphy_new()' fails, we leak 'ops'. Add a new label in the error > handling path to free it in such a case. Thanks. Please add the following tags: Cc: stable@vger.kernel.org > Fixes: 5c22fb85102a7 ("brcmfmac: add wowl gtk rekeying offload support") Acked-by: Arend van Spriel > Signed-off-by: Christophe JAILLET > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-)