Re: [dpdk-dev] [PATCH] pdump: fix possible mbuf leak

2017-11-11 Thread Thomas Monjalon
10/11/2017 14:55, Ilya V. Matveychikov: > If pdump_pktmbuf_copy_data() fails it's possible to have segment leak > as rte_pktmbuf_free() only handles m_dup chain but not the seg just > allocated and yet not chained. > > Fixes: 278f945402c5 ("pdump: add new library for packet capture") > Cc: reshma.

[dpdk-dev] [PATCH] pdump: fix possible mbuf leak

2017-11-10 Thread Ilya V. Matveychikov
If pdump_pktmbuf_copy_data() fails it's possible to have segment leak as rte_pktmbuf_free() only handles m_dup chain but not the seg just allocated and yet not chained. Fixes: 278f945402c5 ("pdump: add new library for packet capture") Cc: reshma.pat...@intel.com Signed-off-by: Ilya V. Matveychiko

Re: [dpdk-dev] [PATCH] pdump: fix possible mbuf leak

2017-11-09 Thread Ferruh Yigit
On 11/9/2017 5:49 AM, Ilya Matveychikov wrote: > If pdump_pktmbuf_copy_data() fails it's possible to have segment leak > as rte_pktmbuf_free() only handles m_dup chain but not the seg just > allocated and yet not chained. > > Fixes: 278f9454 ("pdump: add new library for packet capture") > Signed-o

[dpdk-dev] [PATCH] pdump: fix possible mbuf leak

2017-11-09 Thread Ilya Matveychikov
If pdump_pktmbuf_copy_data() fails it's possible to have segment leak as rte_pktmbuf_free() only handles m_dup chain but not the seg just allocated and yet not chained. Fixes: 278f9454 ("pdump: add new library for packet capture") Signed-off-by: Ilya V. Matveychikov --- lib/librte_pdump/rte_pdum