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

2015-08-05 Thread Iremonger, Bernard
Hi Thomas, > What is the root cause of the crash? > I guess it is rte_free(eth_dev->data->mac_addrs) because eth_dev is NULL. Yes, that is the roor cause. > > [...] > > -err: > > - rte_free(pci_dev); > > - rte_free(internals); > > +err4: > > rte_free(eth_dev->data->mac_addrs); > > -

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

2015-08-04 Thread Thomas Monjalon
2015-08-04 16:26, Bernard Iremonger: > if the name parameter to rte_eth_bond_create() was NULL, > there was a segmentation fault. What is the root cause of the crash? I guess it is rte_free(eth_dev->data->mac_addrs) because eth_dev is NULL. [...] > -err: > - rte_free(pci_dev); > - rte_fre

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

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