[dpdk-dev] [PATCH 3/3] lpm: fix missing free of lpm

2016-03-16 Thread Christian Ehrhardt
Hi, looking at it I think we have intersections but also parts of yours that I missed. More than that while applying your changes I found other potential use-after free cases. I'll wrap that all up together in a v3 of my series. Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd O

[dpdk-dev] [PATCH 3/3] lpm: fix missing free of lpm

2016-03-16 Thread Olivier MATZ
Hi Christian, On 03/16/2016 01:33 PM, Christian Ehrhardt wrote: > Fixing lpm6 regarding a similar issue showed that that in rte_lpm_free lpm > might not be freed if it didn't find a te (early return) > > Acked-by: Bruce Richardson > Signed-off-by: Christian Ehrhardt > --- > lib/librte_lpm/rte_

[dpdk-dev] [PATCH 3/3] lpm: fix missing free of lpm

2016-03-16 Thread Christian Ehrhardt
Fixing lpm6 regarding a similar issue showed that that in rte_lpm_free lpm might not be freed if it didn't find a te (early return) Acked-by: Bruce Richardson Signed-off-by: Christian Ehrhardt --- lib/librte_lpm/rte_lpm.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git