mbox series

[v2,0/3] kvargs fixes

Message ID 20200327080955.19571-1-olivier.matz@6wind.com (mailing list archive)
Headers
Series kvargs fixes |

Message

Olivier Matz March 27, 2020, 8:09 a.m. UTC 
  This patchset fixes a buffer overflow in kvargs when parsing an invalid
string, and fixes the kvargs unit tests.

Olivier Matz (2):
  tests/kvargs: fix to consider empty elements as valid
  tests/kvargs: fix check of invalid cases

Yunjian Wang (1):
  kvargs: fix a heap buffer overflow when parsing list

 app/test/test_kvargs.c         | 40 +++++++++++++++++++++++++++++++---
 lib/librte_kvargs/rte_kvargs.c |  2 ++
 2 files changed, 39 insertions(+), 3 deletions(-)

Comments

David Marchand March 27, 2020, 3:16 p.m. UTC | #1 
On Fri, Mar 27, 2020 at 9:10 AM Olivier Matz <olivier.matz@6wind.com> wrote:
>
> This patchset fixes a buffer overflow in kvargs when parsing an invalid
> string, and fixes the kvargs unit tests.
>
> Olivier Matz (2):
>   tests/kvargs: fix to consider empty elements as valid
>   tests/kvargs: fix check of invalid cases
>
> Yunjian Wang (1):
>   kvargs: fix a heap buffer overflow when parsing list

For the series,
Reviewed-by: David Marchand <david.marchand@redhat.com>
David Marchand March 27, 2020, 5:12 p.m. UTC | #2 
On Fri, Mar 27, 2020 at 4:16 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> On Fri, Mar 27, 2020 at 9:10 AM Olivier Matz <olivier.matz@6wind.com> wrote:
> >
> > This patchset fixes a buffer overflow in kvargs when parsing an invalid
> > string, and fixes the kvargs unit tests.
> >
> > Olivier Matz (2):
> >   tests/kvargs: fix to consider empty elements as valid
> >   tests/kvargs: fix check of invalid cases
> >
> > Yunjian Wang (1):
> >   kvargs: fix a heap buffer overflow when parsing list
>
> For the series,
> Reviewed-by: David Marchand <david.marchand@redhat.com>

Applied, thanks.