NFC on Linux
 help / color / mirror / Atom feed
* Re: [PATCH] selftests: nci: Fix wrong size of status variables in nci_dev test
       [not found] <20260710112300.512433-1-thuth@redhat.com>
@ 2026-07-21 20:56 ` Jakub Kicinski
  0 siblings, 0 replies; only message in thread
From: Jakub Kicinski @ 2026-07-21 20:56 UTC (permalink / raw)
  To: Thomas Huth, oe-linux-nfc
  Cc: Bongsu Jeon, netdev, Shuah Khan, linux-kselftest, linux-kernel

On Fri, 10 Jul 2026 13:23:00 +0200 Thomas Huth wrote:
> From: Thomas Huth <thuth@redhat.com>
> 
> pthread_join() stores the thread's return value (a "void *", i.e.
> 8 bytes on 64 bit computers) into the address of "status", but the
> "status" variable is declared as "int" with only 4 bytes. The extra
> four bytes clobber whatever is adjacent on the stack, which could
> silently corrupt other local variables. Use "intptr_t" to declare
> the "status" variables with the correct size.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>

This should go via nfc, I just sent a patch to update MAINTAINERS.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2026-07-21 20:56 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20260710112300.512433-1-thuth@redhat.com>
2026-07-21 20:56 ` [PATCH] selftests: nci: Fix wrong size of status variables in nci_dev test Jakub Kicinski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox