Re: [dpdk-dev] [PATCH 5/5] cfgfile: increase local buffer size for max name and value

2017-03-09 Thread Wiles, Keith
> On Mar 9, 2017, at 9:16 AM, Legacy, Allain > wrote: > >> -Original Message- >> From: Wiles, Keith [mailto:keith.wi...@intel.com] >> Sent: Thursday, March 09, 2017 8:46 AM >> Would this change still cause a failure and memory over write if the user >> decides to have very large string.

Re: [dpdk-dev] [PATCH 5/5] cfgfile: increase local buffer size for max name and value

2017-03-09 Thread Legacy, Allain
> -Original Message- > From: Wiles, Keith [mailto:keith.wi...@intel.com] > Sent: Thursday, March 09, 2017 8:46 AM > Would this change still cause a failure and memory over write if the user > decides to have very large string. Does the code check the lengths to make > sure they are valid an

Re: [dpdk-dev] [PATCH 5/5] cfgfile: increase local buffer size for max name and value

2017-03-09 Thread Wiles, Keith
> On Mar 2, 2017, at 1:29 PM, Allain Legacy wrote: > > From: Joseph Richard > > When parsing a ini file with a "key = value" line that has both "key" and > "value" sized to the maximum allowed length causes a parsing failure. The > internal "buffer" variable should be sized at least as large

[dpdk-dev] [PATCH 5/5] cfgfile: increase local buffer size for max name and value

2017-03-02 Thread Allain Legacy
From: Joseph Richard When parsing a ini file with a "key = value" line that has both "key" and "value" sized to the maximum allowed length causes a parsing failure. The internal "buffer" variable should be sized at least as large as the maximum for both fields. This commit updates the local arr