From: HongBo Zheng <[email protected]>

Return value of a function 'rte_eth_macaddr_get' called at
l3fwd_eth_dev_port_setup is not checked, but it is usually
checked for this function.

This patch fix this problem.

Fixes: a65bf3d724df ("examples/l3fwd: add ethdev setup based on eventdev")
Cc: [email protected]

Signed-off-by: HongBo Zheng <[email protected]>
Signed-off-by: Min Hu (Connor) <[email protected]>
---
 examples/l3fwd/l3fwd_event.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/examples/l3fwd/l3fwd_event.c b/examples/l3fwd/l3fwd_event.c
index 4d31593..7f704f9 100644
--- a/examples/l3fwd/l3fwd_event.c
+++ b/examples/l3fwd/l3fwd_event.c
@@ -105,7 +105,11 @@ l3fwd_eth_dev_port_setup(struct rte_eth_conf *port_conf)
                                 "Cannot adjust number of descriptors: err=%d, "
                                 "port=%d\n", ret, port_id);
 
-               rte_eth_macaddr_get(port_id, &ports_eth_addr[port_id]);
+               ret = rte_eth_macaddr_get(port_id, &ports_eth_addr[port_id]);
+               if (ret < 0)
+                       rte_exit(EXIT_FAILURE,
+                                "Cannot get MAC address: err=%d, port=%d\n",
+                                ret, port_id);
                print_ethaddr(" Address:", &ports_eth_addr[port_id]);
                printf(", ");
                print_ethaddr("Destination:",
-- 
2.7.4

Reply via email to