1c29f2a
From patchwork Fri Oct 25 20:42:42 2019
1c29f2a
Content-Type: text/plain; charset="utf-8"
1c29f2a
MIME-Version: 1.0
1c29f2a
Content-Transfer-Encoding: 7bit
1c29f2a
X-Patchwork-Submitter: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
1c29f2a
X-Patchwork-Id: 1184451
1c29f2a
X-Patchwork-Delegate: davem@davemloft.net
1c29f2a
Return-Path: <netdev-owner@vger.kernel.org>
1c29f2a
X-Original-To: patchwork-incoming-netdev@ozlabs.org
1c29f2a
Delivered-To: patchwork-incoming-netdev@ozlabs.org
1c29f2a
Authentication-Results: ozlabs.org; spf=none (no SPF record)
1c29f2a
 smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67;
1c29f2a
 helo=vger.kernel.org;
1c29f2a
 envelope-from=netdev-owner@vger.kernel.org;
1c29f2a
 receiver=<UNKNOWN>)
1c29f2a
Authentication-Results: ozlabs.org;
1c29f2a
 dmarc=fail (p=none dis=none) header.from=intel.com
1c29f2a
Received: from vger.kernel.org (vger.kernel.org [209.132.180.67])
1c29f2a
 by ozlabs.org (Postfix) with ESMTP id 470GJz096Tz9sPf
1c29f2a
 for <patchwork-incoming-netdev@ozlabs.org>;
1c29f2a
 Sat, 26 Oct 2019 07:42:59 +1100 (AEDT)
1c29f2a
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
1c29f2a
 id S1728655AbfJYUmz (ORCPT
1c29f2a
 <rfc822;patchwork-incoming-netdev@ozlabs.org>);
1c29f2a
 Fri, 25 Oct 2019 16:42:55 -0400
1c29f2a
Received: from mga11.intel.com ([192.55.52.93]:22523 "EHLO mga11.intel.com"
1c29f2a
 rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
1c29f2a
 id S1728604AbfJYUmq (ORCPT <rfc822;netdev@vger.kernel.org>);
1c29f2a
 Fri, 25 Oct 2019 16:42:46 -0400
1c29f2a
X-Amp-Result: SKIPPED(no attachment in message)
1c29f2a
X-Amp-File-Uploaded: False
1c29f2a
Received: from orsmga006.jf.intel.com ([10.7.209.51])
1c29f2a
 by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
1c29f2a
 25 Oct 2019 13:42:46 -0700
1c29f2a
X-ExtLoop1: 1
1c29f2a
X-IronPort-AV: E=Sophos;i="5.68,229,1569308400"; d="scan'208";a="202713981"
1c29f2a
Received: from jtkirshe-desk1.jf.intel.com ([134.134.177.96])
1c29f2a
 by orsmga006.jf.intel.com with ESMTP; 25 Oct 2019 13:42:44 -0700
1c29f2a
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
1c29f2a
To: davem@davemloft.net
1c29f2a
Cc: Navid Emamdoost <navid.emamdoost@gmail.com>,
1c29f2a
 netdev@vger.kernel.org, nhorman@redhat.com, sassmann@redhat.com,
1c29f2a
 Andrew Bowers <andrewx.bowers@intel.com>,
1c29f2a
 Jeff Kirsher <jeffrey.t.kirsher@intel.com>
1c29f2a
Subject: [net-next v2 9/9] i40e: prevent memory leak in i40e_setup_macvlans
1c29f2a
Date: Fri, 25 Oct 2019 13:42:42 -0700
1c29f2a
Message-Id: <20191025204242.10535-10-jeffrey.t.kirsher@intel.com>
1c29f2a
X-Mailer: git-send-email 2.21.0
1c29f2a
In-Reply-To: <20191025204242.10535-1-jeffrey.t.kirsher@intel.com>
1c29f2a
References: <20191025204242.10535-1-jeffrey.t.kirsher@intel.com>
1c29f2a
MIME-Version: 1.0
1c29f2a
Sender: netdev-owner@vger.kernel.org
1c29f2a
Precedence: bulk
1c29f2a
List-ID: <netdev.vger.kernel.org>
1c29f2a
X-Mailing-List: netdev@vger.kernel.org
1c29f2a
1c29f2a
From: Navid Emamdoost <navid.emamdoost@gmail.com>
1c29f2a
1c29f2a
In i40e_setup_macvlans if i40e_setup_channel fails the allocated memory
1c29f2a
for ch should be released.
1c29f2a
1c29f2a
Signed-off-by: Navid Emamdoost <navid.emamdoost@gmail.com>
1c29f2a
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
1c29f2a
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
1c29f2a
---
1c29f2a
 drivers/net/ethernet/intel/i40e/i40e_main.c | 1 +
1c29f2a
 1 file changed, 1 insertion(+)
1c29f2a
1c29f2a
diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
1c29f2a
index 2e4df0bd8d37..141575ada588 100644
1c29f2a
--- a/drivers/net/ethernet/intel/i40e/i40e_main.c
1c29f2a
+++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
1c29f2a
@@ -7187,6 +7187,7 @@ static int i40e_setup_macvlans(struct i40e_vsi *vsi, u16 macvlan_cnt, u16 qcnt,
1c29f2a
 		ch->num_queue_pairs = qcnt;
1c29f2a
 		if (!i40e_setup_channel(pf, vsi, ch)) {
1c29f2a
 			ret = -EINVAL;
1c29f2a
+			kfree(ch);
1c29f2a
 			goto err_free;
1c29f2a
 		}
1c29f2a
 		ch->parent_vsi = vsi;