Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Kevin Traynor
On 01/31/2019 03:55 PM, Burakov, Anatoly wrote: > On 31-Jan-19 3:04 PM, Thomas Monjalon wrote: >> 31/01/2019 15:15, Kevin Traynor: >>> On 01/31/2019 02:10 PM, Burakov, Anatoly wrote: On 31-Jan-19 11:21 AM, Kevin Traynor wrote: > On 01/10/2019 01:38 PM, Anatoly Burakov wrote: >> Current

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Burakov, Anatoly
On 31-Jan-19 3:04 PM, Thomas Monjalon wrote: 31/01/2019 15:15, Kevin Traynor: On 01/31/2019 02:10 PM, Burakov, Anatoly wrote: On 31-Jan-19 11:21 AM, Kevin Traynor wrote: On 01/10/2019 01:38 PM, Anatoly Burakov wrote: Currently, we use strdup in a few places to store command-line parameter val

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Thomas Monjalon
31/01/2019 15:15, Kevin Traynor: > On 01/31/2019 02:10 PM, Burakov, Anatoly wrote: > > On 31-Jan-19 11:21 AM, Kevin Traynor wrote: > >> On 01/10/2019 01:38 PM, Anatoly Burakov wrote: > >>> Currently, we use strdup in a few places to store command-line > >>> parameter values for certain internal con

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Kevin Traynor
On 01/31/2019 02:10 PM, Burakov, Anatoly wrote: > On 31-Jan-19 11:21 AM, Kevin Traynor wrote: >> On 01/10/2019 01:38 PM, Anatoly Burakov wrote: >>> Currently, we use strdup in a few places to store command-line >>> parameter values for certain internal config values. There are >>> several issues wi

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Burakov, Anatoly
On 31-Jan-19 11:21 AM, Kevin Traynor wrote: On 01/10/2019 01:38 PM, Anatoly Burakov wrote: Currently, we use strdup in a few places to store command-line parameter values for certain internal config values. There are several issues with that. First of all, they're never freed, so memory ends up

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-31 Thread Kevin Traynor
On 01/10/2019 01:38 PM, Anatoly Burakov wrote: > Currently, we use strdup in a few places to store command-line > parameter values for certain internal config values. There are > several issues with that. > > First of all, they're never freed, so memory ends up leaking > either after EAL exit, or

Re: [dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-14 Thread Thomas Monjalon
10/01/2019 14:38, Anatoly Burakov: > Currently, we use strdup in a few places to store command-line > parameter values for certain internal config values. There are > several issues with that. > > First of all, they're never freed, so memory ends up leaking > either after EAL exit, or when these c

[dpdk-dev] [PATCH] eal: fix strdup usages in internal config

2019-01-10 Thread Anatoly Burakov
Currently, we use strdup in a few places to store command-line parameter values for certain internal config values. There are several issues with that. First of all, they're never freed, so memory ends up leaking either after EAL exit, or when these command-line options are supplied multiple times