forked from rpms/kernel
		
	
		
			
				
	
	
		
			110 lines
		
	
	
		
			5.5 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			110 lines
		
	
	
		
			5.5 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
From mboxrd@z Thu Jan  1 00:00:00 1970
 | 
						|
Return-Path: <SRS0=wkwR=X5=vger.kernel.org=netdev-owner@kernel.org>
 | 
						|
X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on
 | 
						|
	aws-us-west-2-korg-lkml-1.web.codeaurora.org
 | 
						|
X-Spam-Level: 
 | 
						|
X-Spam-Status: No, score=-9.5 required=3.0 tests=DKIM_ADSP_CUSTOM_MED,
 | 
						|
	DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,
 | 
						|
	HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,
 | 
						|
	SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable
 | 
						|
	autolearn_force=no version=3.4.0
 | 
						|
Received: from mail.kernel.org (mail.kernel.org [198.145.29.99])
 | 
						|
	by smtp.lore.kernel.org (Postfix) with ESMTP id 973EDC47404
 | 
						|
	for <netdev@archiver.kernel.org>; Fri,  4 Oct 2019 20:09:06 +0000 (UTC)
 | 
						|
Received: from vger.kernel.org (vger.kernel.org [209.132.180.67])
 | 
						|
	by mail.kernel.org (Postfix) with ESMTP id 700B62133F
 | 
						|
	for <netdev@archiver.kernel.org>; Fri,  4 Oct 2019 20:09:06 +0000 (UTC)
 | 
						|
Authentication-Results: mail.kernel.org;
 | 
						|
	dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="kWRjsMO7"
 | 
						|
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
 | 
						|
        id S1731310AbfJDUJE (ORCPT <rfc822;netdev@archiver.kernel.org>);
 | 
						|
        Fri, 4 Oct 2019 16:09:04 -0400
 | 
						|
Received: from mail-io1-f68.google.com ([209.85.166.68]:42107 "EHLO
 | 
						|
        mail-io1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
 | 
						|
        with ESMTP id S1729079AbfJDUJD (ORCPT
 | 
						|
        <rfc822;netdev@vger.kernel.org>); Fri, 4 Oct 2019 16:09:03 -0400
 | 
						|
Received: by mail-io1-f68.google.com with SMTP id n197so16151518iod.9;
 | 
						|
        Fri, 04 Oct 2019 13:09:03 -0700 (PDT)
 | 
						|
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 | 
						|
        d=gmail.com; s=20161025;
 | 
						|
        h=from:to:cc:subject:date:message-id;
 | 
						|
        bh=TzGHd63YhhSYgCiRZ9tl0C+QqBhc3B9t1YurdXMHHEk=;
 | 
						|
        b=kWRjsMO7kWGMm/IT1zcaML7wX7Qq9Yq8WRm+RABc1yvFRuSkg1MJFKvuhIADpR86Sk
 | 
						|
         vk1W0Z5v0fSKpGUd8/OWzfGGYUmUKyVV0GCdL13GMMOSK9DpPSyvyt77eG5WmY3ZmctQ
 | 
						|
         TGC8Cmmzyp9qWwicYV847MUo/KmeyE8yz2h7Smb7WOcrkh2TV9S23hinrXd+nv5M0ZyF
 | 
						|
         ASEYU02I8rRxFXPCLp8D/KgP4shuTzi8Fg2/Hk1WrnV7b+j7LB2iDWk/cyp1qilFIKMU
 | 
						|
         ZBR7mheI1uaHirJEtlL1lGs5B+v25rI82daNM0pAfgJ1uilLnWPeFNCSgjxJjo/1l4rp
 | 
						|
         Y9qg==
 | 
						|
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 | 
						|
        d=1e100.net; s=20161025;
 | 
						|
        h=x-gm-message-state:from:to:cc:subject:date:message-id;
 | 
						|
        bh=TzGHd63YhhSYgCiRZ9tl0C+QqBhc3B9t1YurdXMHHEk=;
 | 
						|
        b=i14dfaCVcowVdtunic+1Wp2wGV+1ZNLuoXJwp7E15v+1eEJZwFN3kPpygTaLhCxUcK
 | 
						|
         GwKISgBQGzHrbHDnyg1HwrLXM02Ldx63K+ocW92bWoiabvGepNaNkKLLjh1Cc9Z5Udr5
 | 
						|
         FaRfo4jqaLUlKwSETn1kU+GSCbHc7OjVJMEAosWc4bRRpy2Z+vcoUwHuCG5OaNg6EQsh
 | 
						|
         rYclOJRJfMAu448q1becWoBHIpImXDm50NiWfQtEoUOj5o22/QPRUwaUeIC5LrB/sOiI
 | 
						|
         ZDVFoRHgEjTok93ZXTlcDfOXok4PcETTA6aDez2v7cwSd2y9Z9PI3Ng1mnPK/vmvqmXW
 | 
						|
         iQZw==
 | 
						|
X-Gm-Message-State: APjAAAXV8P3/lHsEKg8iHeiLr3dvIg5sDmZQ5/1qeeMO2VMTctGtUgtN
 | 
						|
        UfU23ad10Ce+u/Moful94ba6TP6V/uo=
 | 
						|
X-Google-Smtp-Source: APXvYqzID3S043QvcjXSlnz2Lfed2XSsv1r0x047tSPe8/gvBVD1NyJxtMQ/ZV89X4qzhOAjTbC2WA==
 | 
						|
X-Received: by 2002:a92:603:: with SMTP id x3mr17481056ilg.295.1570219742776;
 | 
						|
        Fri, 04 Oct 2019 13:09:02 -0700 (PDT)
 | 
						|
Received: from cs-dulles.cs.umn.edu (cs-dulles.cs.umn.edu. [128.101.35.54])
 | 
						|
        by smtp.googlemail.com with ESMTPSA id i67sm4267112ilf.84.2019.10.04.13.09.02
 | 
						|
        (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
 | 
						|
        Fri, 04 Oct 2019 13:09:02 -0700 (PDT)
 | 
						|
From:   Navid Emamdoost <navid.emamdoost@gmail.com>
 | 
						|
Cc:     emamd001@umn.edu, kjlu@umn.edu, smccaman@umn.edu,
 | 
						|
        Navid Emamdoost <navid.emamdoost@gmail.com>,
 | 
						|
        Amitkumar Karwar <amitkarwar@gmail.com>,
 | 
						|
        Nishant Sarmukadam <nishants@marvell.com>,
 | 
						|
        Ganapathi Bhat <gbhat@marvell.com>,
 | 
						|
        Xinming Hu <huxinming820@gmail.com>,
 | 
						|
        Kalle Valo <kvalo@codeaurora.org>,
 | 
						|
        "David S. Miller" <davem@davemloft.net>,
 | 
						|
        linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
 | 
						|
        linux-kernel@vger.kernel.org
 | 
						|
Subject: [PATCH] mwifiex: pcie: Fix memory leak in mwifiex_pcie_alloc_cmdrsp_buf
 | 
						|
Date:   Fri,  4 Oct 2019 15:08:52 -0500
 | 
						|
Message-Id: <20191004200853.23353-1-navid.emamdoost@gmail.com>
 | 
						|
X-Mailer: git-send-email 2.17.1
 | 
						|
To:     unlisted-recipients:; (no To-header on input)
 | 
						|
Sender: netdev-owner@vger.kernel.org
 | 
						|
Precedence: bulk
 | 
						|
List-ID: <netdev.vger.kernel.org>
 | 
						|
X-Mailing-List: netdev@vger.kernel.org
 | 
						|
Archived-At: <https://lore.kernel.org/netdev/20191004200853.23353-1-navid.emamdoost@gmail.com/>
 | 
						|
List-Archive: <https://lore.kernel.org/netdev/>
 | 
						|
List-Post: <mailto:netdev@vger.kernel.org>
 | 
						|
 | 
						|
In mwifiex_pcie_alloc_cmdrsp_buf, a new skb is allocated which should be
 | 
						|
released if mwifiex_map_pci_memory() fails. The release is added.
 | 
						|
 | 
						|
Fixes: fc3314609047 ("mwifiex: use pci_alloc/free_consistent APIs for PCIe")
 | 
						|
Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
 | 
						|
---
 | 
						|
 drivers/net/wireless/marvell/mwifiex/pcie.c | 4 +++-
 | 
						|
 1 file changed, 3 insertions(+), 1 deletion(-)
 | 
						|
 | 
						|
diff --git a/drivers/net/wireless/marvell/mwifiex/pcie.c b/drivers/net/wireless/marvell/mwifiex/pcie.c
 | 
						|
index eff06d59e9df..1578eaa071bd 100644
 | 
						|
--- a/drivers/net/wireless/marvell/mwifiex/pcie.c
 | 
						|
+++ b/drivers/net/wireless/marvell/mwifiex/pcie.c
 | 
						|
@@ -1029,8 +1029,10 @@ static int mwifiex_pcie_alloc_cmdrsp_buf(struct mwifiex_adapter *adapter)
 | 
						|
 	}
 | 
						|
 	skb_put(skb, MWIFIEX_UPLD_SIZE);
 | 
						|
 	if (mwifiex_map_pci_memory(adapter, skb, MWIFIEX_UPLD_SIZE,
 | 
						|
-				   PCI_DMA_FROMDEVICE))
 | 
						|
+				   PCI_DMA_FROMDEVICE)) {
 | 
						|
+		kfree_skb(skb);
 | 
						|
 		return -1;
 | 
						|
+	}
 | 
						|
 
 | 
						|
 	card->cmdrsp_buf = skb;
 | 
						|
 
 | 
						|
-- 
 | 
						|
2.17.1
 | 
						|
 | 
						|
 |