All of lore.kernel.org
 help / color / mirror / Atom feed
From: Breno Leitao <leitao@debian.org>
To: Simon Horman <horms@kernel.org>
Cc: Jakub Kicinski <kuba@kernel.org>,
	davem@davemloft.net,  netdev@vger.kernel.org,
	edumazet@google.com, pabeni@redhat.com, andrew+netdev@lunn.ch
Subject: Re: [PATCH net] netconsole: avoid OOB reads, msg is not nul-terminated
Date: Mon, 2 Mar 2026 01:47:18 -0800	[thread overview]
Message-ID: <aaVcivgYdauX6yGk@gmail.com> (raw)
In-Reply-To: <aZsrH23rOB0Z4_fz@horms.kernel.org>

On Sun, Feb 22, 2026 at 04:13:19PM +0000, Simon Horman wrote:
> On Thu, Feb 19, 2026 at 11:50:21AM -0800, Jakub Kicinski wrote:
> > msg passed to netconsole from the console subsystem is not guaranteed
> > to be nul-terminated. Before recent
> > commit 7eab73b18630 ("netconsole: convert to NBCON console infrastructure")
> > the message would be placed in printk_shared_pbufs, a static global
> > buffer, so KASAN had harder time catching OOB accesses. Now we see:
> > 
> >     printk: console [netcon_ext0] enabled
> >     BUG: KASAN: slab-out-of-bounds in string+0x1f7/0x240
> >     Read of size 1 at addr ffff88813b6d4c00 by task pr/netcon_ext0/594
> > 
> >     CPU: 65 UID: 0 PID: 594 Comm: pr/netcon_ext0 Not tainted 6.19.0-11754-g4246fd6547c9
> >     Call Trace:
> >      kasan_report+0xe4/0x120
> >      string+0x1f7/0x240
> >      vsnprintf+0x655/0xba0
> >      scnprintf+0xba/0x120
> >      netconsole_write+0x3fe/0xa10
> >      nbcon_emit_next_record+0x46e/0x860
> >      nbcon_kthread_func+0x623/0x750
> > 
> >     Allocated by task 1:
> >      nbcon_alloc+0x1ea/0x450
> >      register_console+0x26b/0xe10
> >      init_netconsole+0xbb0/0xda0
> > 
> >     The buggy address belongs to the object at ffff88813b6d4000
> >                 which belongs to the cache kmalloc-4k of size 4096
> >     The buggy address is located 0 bytes to the right of
> >                 allocated 3072-byte region [ffff88813b6d4000, ffff88813b6d4c00)
> > 
> > Fixes: c62c0a17f9b7 ("netconsole: Append kernel version to message")
> > Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> 
> Reviewed-by: Simon Horman <horms@kernel.org>

Thanks for the fix and review while I was out.

--breno

  reply	other threads:[~2026-03-02  9:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-19 19:50 [PATCH net] netconsole: avoid OOB reads, msg is not nul-terminated Jakub Kicinski
2026-02-22 16:13 ` Simon Horman
2026-03-02  9:47   ` Breno Leitao [this message]
2026-02-24  9:50 ` patchwork-bot+netdevbpf

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:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

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

  git send-email \
    --in-reply-to=aaVcivgYdauX6yGk@gmail.com \
    --to=leitao@debian.org \
    --cc=andrew+netdev@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.