[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <zt6p77wtnc2rrw3hko3ppl2sruy64fhrz2mhgyahvpjuzitpga@tpjsfhxmwrep>
Date: Mon, 12 Jan 2026 13:38:28 +0100
From: Michal Koutný <mkoutny@...e.com>
To: Andrei Vagin <avagin@...gle.com>
Cc: Kees Cook <kees@...nel.org>, linux-kernel@...r.kernel.org,
linux-fsdevel@...r.kernel.org, linux-mm@...ck.org, criu@...ts.linux.dev,
Andrew Morton <akpm@...ux-foundation.org>, Chen Ridong <chenridong@...wei.com>,
Christian Brauner <brauner@...nel.org>, David Hildenbrand <david@...nel.org>,
Eric Biederman <ebiederm@...ssion.com>, Lorenzo Stoakes <lorenzo.stoakes@...cle.com>,
Mark Brown <broonie@...nel.org>, Max Filippov <jcmvbkbc@...il.com>
Subject: Re: [PATCH 1/3] binfmt_elf_fdpic: fix AUXV size calculation for
ELF_HWCAP3 and ELF_HWCAP4
On Thu, Jan 08, 2026 at 05:07:46AM +0000, Andrei Vagin <avagin@...gle.com> wrote:
> Commit 4e6e8c2b757f ("binfmt_elf: Wire up AT_HWCAP3 at AT_HWCAP4") added
> support for AT_HWCAP3 and AT_HWCAP4, but it missed updating the AUX
> vector size calculation in create_elf_fdpic_tables() and
> AT_VECTOR_SIZE_BASE in include/linux/auxvec.h.
>
> Similar to the fix for ELF_HWCAP2 in commit c6a09e342f8e
> ("binfmt_elf_fdpic: fix AUXV size calculation when ELF_HWCAP2 is defined"),
> this omission leads to a mismatch between the reserved space and the
> actual number of AUX entries, eventually triggering a kernel BUG_ON(csp != sp).
>
> Fix this by incrementing nitems when ELF_HWCAP3 or ELF_HWCAP4 are defined
> and updating AT_VECTOR_SIZE_BASE.
>
> Cc: Mark Brown <broonie@...nel.org>
> Cc: Max Filippov <jcmvbkbc@...il.com>
> Fixes: 4e6e8c2b757f ("binfmt_elf: Wire up AT_HWCAP3 at AT_HWCAP4")
> Signed-off-by: Andrei Vagin <avagin@...gle.com>
> ---
> fs/binfmt_elf_fdpic.c | 6 ++++++
> include/linux/auxvec.h | 2 +-
> 2 files changed, 7 insertions(+), 1 deletion(-)
Good catch.
Reviewed-by: Michal Koutný <mkoutny@...e.com>
Download attachment "signature.asc" of type "application/pgp-signature" (266 bytes)
Powered by blists - more mailing lists