Re: [dpdk-dev] [PATCH v2] eal: fix resource leak

2017-10-05 Thread Thomas Monjalon
04/10/2017 21:24, Ferruh Yigit: > On 9/22/2017 3:48 PM, Daniel Mrzyglod wrote: > > Memory allocated in strdup is not free. > > > > Coverity issue: 143257 > > Fixes: d8a2bc71dfc2 ("log: remove app path from syslog id") > > Cc: tho...@monjalon.net > > > > Signed-off-by: Daniel Mrzyglod > > --- > T

Re: [dpdk-dev] [PATCH v2] eal: fix resource leak

2017-10-04 Thread Ferruh Yigit
On 9/22/2017 3:48 PM, Daniel Mrzyglod wrote: > Memory allocated in strdup is not free. > > Coverity issue: 143257 > Fixes: d8a2bc71dfc2 ("log: remove app path from syslog id") > Cc: tho...@monjalon.net > > Signed-off-by: Daniel Mrzyglod > --- > v2: > * Fix due to compilation errors > > lib/lib

Re: [dpdk-dev] [PATCH v2] eal: fix resource leak

2017-10-02 Thread Jastrzebski, MichalX K
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Daniel Mrzyglod > Sent: Friday, September 22, 2017 4:48 PM > To: tho...@monjalon.net > Cc: dev@dpdk.org; Mrzyglod, DanielX T > Subject: [dpdk-dev] [PATCH v2] eal: fix resource leak > > Me

[dpdk-dev] [PATCH v2] eal: fix resource leak

2017-09-22 Thread Daniel Mrzyglod
Memory allocated in strdup is not free. Coverity issue: 143257 Fixes: d8a2bc71dfc2 ("log: remove app path from syslog id") Cc: tho...@monjalon.net Signed-off-by: Daniel Mrzyglod --- v2: * Fix due to compilation errors lib/librte_eal/linuxapp/eal/eal.c | 18 +- 1 file changed, 1

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-20 Thread Tetsuya Mukawa
On 2016/06/17 21:28, David Marchand wrote: > On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote: >> This patch fixes resource leak of pci_uio_map_secondary(). >> If pci_map_resource() succeeds but mapped address is different from an >> address primary process mapped, this should be error. >> Th

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-20 Thread Thomas Monjalon
2016-06-20 11:19, Tetsuya Mukawa: > On 2016/06/17 21:28, David Marchand wrote: > > On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa > > wrote: > >> This patch fixes resource leak of pci_uio_map_secondary(). > >> If pci_map_resource() succeeds but mapped address is different from an > >> address pr

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-17 Thread David Marchand
On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote: > This patch fixes resource leak of pci_uio_map_secondary(). > If pci_map_resource() succeeds but mapped address is different from an > address primary process mapped, this should be error. > Then the addresses secondary process mapped should

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-16 Thread Tetsuya Mukawa
This patch fixes resource leak of pci_uio_map_secondary(). If pci_map_resource() succeeds but mapped address is different from an address primary process mapped, this should be error. Then the addresses secondary process mapped should be freed. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/co