cc the EAL maintainer, David Marchand. On Mon, Feb 20, 2017 at 10:04:47PM +0800, [email protected] wrote: > From: Huanle Han <[email protected]> > > rte_eal_dev_detach() didn't remove dev from dev_device_list > after free the dev. So the following attached dev wrote to > the freed memory (tailq entry of previous dev) in below stack: > > == Invalid write of size 8 > == at 0x43A9CE: rte_eal_device_insert (eal_common_dev.c:71) > == by 0x42ED9E: pci_scan_one (eal_pci.c:365) > == by 0x42EF4D: pci_update_device (eal_pci.c:391) > == by 0x437F59: rte_eal_pci_probe_one (eal_common_pci.c:357) > == by 0x43AB16: rte_eal_dev_attach (eal_common_dev.c:117) > == by 0x45B3AA: rte_eth_dev_attach (rte_ethdev.c:489) > == ... > > Signed-off-by: Huanle Han <[email protected]>
The two minor nits also apply here, besides that, Reviewed-by: Yuanhan Liu <[email protected]> --yliu
