[dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in rte_eth_bond_create()

2015-08-05 Thread Thomas Monjalon
2015-08-05 13:36, Bernard Iremonger: > if the name parameter to rte_eth_bond_create() was NULL, > there was a segmentation fault because eth_dev was also NULL. You also add error handling of mac_addrs alloc and release_port(). It deserves to be said in this commit message.

[dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in rte_eth_bond_create()

2015-08-05 Thread Iremonger, Bernard
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, August 5, 2015 2:35 PM > To: Iremonger, Bernard > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in > rte_eth_bo

[dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in rte_eth_bond_create()

2015-08-05 Thread Bernard Iremonger
if the name parameter to rte_eth_bond_create() was NULL, there was a segmentation fault because eth_dev was also NULL. Signed-off-by: Bernard Iremonger --- drivers/net/bonding/rte_eth_bond_api.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git

[dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in rte_eth_bond_create()

2015-08-05 Thread Iremonger, Bernard
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, August 5, 2015 2:16 PM > To: Iremonger, Bernard > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in > rte_eth_bo

[dpdk-dev] [PATCH v2 1/1] bonding: fix error handling in rte_eth_bond_create()

2015-08-05 Thread Liu, Yong
Tested-by: Marvin Liu > -Original Message- > From: Iremonger, Bernard > Sent: Wednesday, August 05, 2015 8:37 PM > To: dev at dpdk.org > Cc: Jastrzebski, MichalX K; Liu, Yong; Iremonger, Bernard > Subject: [PATCH v2 1/1] bonding: fix error handling in > rte_eth_bond_create() > > if the