From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Daly Subject: [PATCH] net/bonding: check return value in bond_ethdev_configure Date: Fri, 16 Nov 2018 14:54:27 +0000 Message-ID: <1542380067-140320-1-git-send-email-lee.daly@intel.com> Cc: dev@dpdk.org, Lee Daly To: declan.doherty@intel.com, chas3@att.com Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id 1B06D2B94 for ; Fri, 16 Nov 2018 15:54:35 +0100 (CET) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This patch checks the return value of function rte_eth_bond_8023ad_agg_selection_set() in bond_ethdev_configure for error return value. Signed-off-by: Lee Daly --- drivers/net/bonding/rte_eth_bond_pmd.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c index 156f31c..a7612ae 100644 --- a/drivers/net/bonding/rte_eth_bond_pmd.c +++ b/drivers/net/bonding/rte_eth_bond_pmd.c @@ -3420,9 +3420,14 @@ bond_ethdev_configure(struct rte_eth_dev *dev) "Failed to parse agg selection mode for bonded device %s", name); } - if (internals->mode == BONDING_MODE_8023AD) - rte_eth_bond_8023ad_agg_selection_set(port_id, - agg_mode); + if (internals->mode == BONDING_MODE_8023AD) { + if (rte_eth_bond_8023ad_agg_selection_set(port_id, + agg_mode) < 0) { + RTE_BOND_LOG(ERR, "Invalid args for agg selection" + " set for bonded device %s", name); + return -1; + } + } } /* Parse/add slave ports to bonded device */ -- 2.7.4