[dpdk-dev] [PATCH] lpm: fix memory leak

2016-03-16 Thread Christian Ehrhardt
Hi, I'm done comparing our two patches and just submitted a v3 of my series based on that. I found even more use after free and leaks than we had before. Patch series has grown to 5 patches now. At least my gmail groups subsequent git send-email posts weirdly, let me know if you are in any trouble

[dpdk-dev] [PATCH] lpm: fix memory leak

2016-03-16 Thread Christian Ehrhardt
Thanks Oliver, the bad thing was that I forgot to CC dpdk-dev last friday. I just resubmitted correcting that mistake. I think it should now just be down to the re-review and apply of Bruce. Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd On Wed, Mar 16, 2016 at 2:11 PM, Olivie

[dpdk-dev] [PATCH] lpm: fix memory leak

2016-03-16 Thread Olivier MATZ
On 03/15/2016 01:25 PM, Olivier Matz wrote: > Internal lpm structures are not properly freed. Seen with the > lpm6 autotest. > > Signed-off-by: Olivier Matz > --- > lib/librte_lpm/rte_lpm.c | 3 +++ > lib/librte_lpm/rte_lpm6.c | 1 + > 2 files changed, 4 insertions(+) > Self-nack, Christia

[dpdk-dev] [PATCH] lpm: fix memory leak

2016-03-15 Thread Olivier Matz
Internal lpm structures are not properly freed. Seen with the lpm6 autotest. Signed-off-by: Olivier Matz --- lib/librte_lpm/rte_lpm.c | 3 +++ lib/librte_lpm/rte_lpm6.c | 1 + 2 files changed, 4 insertions(+) diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c index dd62f9b..d28e9