patches for DPDK stable branches
 help / color / mirror / Atom feed
From: David Marchand <>
To: Olivier Matz <>
Cc: dev <>, dpdk stable <>
Subject: Re: [dpdk-stable] [PATCH] kvargs: fix crash when parsing an invalid token on FreeBSD
Date: Wed, 6 May 2020 15:22:40 +0200	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Wed, Apr 29, 2020 at 3:17 PM Olivier Matz <> wrote:
> The behavior of strtok_r() is not the same between GNU libc and FreeBSD
> libc: in the first case, the context is set to "" when the last token is
> returned, while in the second case it is set to NULL.
> On FreeBSD, the current code crashes because we are dereferencing a NULL
> pointer (ctx1). Fix it by first checking if it is NULL. This works with
> both GNU and FreeBSD libc.
> Fixes: ffcf831454a9 ("kvargs: fix buffer overflow when parsing list")
> Cc:
> Signed-off-by: Olivier Matz <>

Tested-by: Zhimin Huang <>

Applied, thanks.

David Marchand

      parent reply	other threads:[~2020-05-06 13:22 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-29 13:17 Olivier Matz
2020-04-30  9:41 ` [dpdk-stable] [dpdk-dev] " Huang, ZhiminX
2020-05-06 13:22 ` David Marchand [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).