From: Ferruh Yigit <ferruh.yigit@intel.com>
Date: Tue, 23 Apr 2019 15:23:32 +0000 (+0100)
Subject: net/ring: fix return value check
X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=e0474b94f8a36672d66be7408e3f7cf00e302329;hp=70a6aeb4639b633bc322834bb6f07b6c45c7e52b;p=dpdk.git

net/ring: fix return value check

'rte_eth_dev_get_port_by_name()' return value is not checked caught by
coverity, adding return value check.

Coverity issue: 305853
Fixes: 96cb19521147 ("net/ring: use EAL APIs in PMD specific API")
Cc: stable@dpdk.org

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---

diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c
index 979993fd3e..ab963a03c4 100644
--- a/drivers/net/ring/rte_eth_ring.c
+++ b/drivers/net/ring/rte_eth_ring.c
@@ -396,7 +396,11 @@ rte_eth_from_rings(const char *name, struct rte_ring *const rx_queues[],
 		return -1;
 	}
 
-	rte_eth_dev_get_port_by_name(ring_name, &port_id);
+	ret = rte_eth_dev_get_port_by_name(ring_name, &port_id);
+	if (ret) {
+		rte_errno = ENODEV;
+		return -1;
+	}
 
 	return port_id;
 }