From ea1a0737c7692737a644af0486b71e4a392cbca8 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Mon, 22 Sep 2025 15:20:34 +0800 Subject: [PATCH] elf: Don't read beyond .eh_frame section size PR ld/33464 * elf-eh-frame.c (_bfd_elf_parse_eh_frame): Don't read beyond .eh_frame section size. Signed-off-by: H.J. Lu --- bfd/elf-eh-frame.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff -rup binutils-2.43.1.orig/bfd/elf-eh-frame.c binutils-2.43.1/bfd/elf-eh-frame.c --- binutils-2.43.1.orig/bfd/elf-eh-frame.c 2025-10-03 12:00:40.473590498 +0100 +++ binutils-2.43.1/bfd/elf-eh-frame.c 2025-10-03 12:00:59.521264872 +0100 @@ -734,6 +734,7 @@ _bfd_elf_parse_eh_frame (bfd *abfd, stru if (hdr_id == 0) { unsigned int initial_insn_length; + char *null_byte; /* CIE */ this_inf->cie = 1; @@ -750,10 +751,13 @@ _bfd_elf_parse_eh_frame (bfd *abfd, stru REQUIRE (cie->version == 1 || cie->version == 3 || cie->version == 4); - REQUIRE (strlen ((char *) buf) < sizeof (cie->augmentation)); + null_byte = memchr ((char *) buf, 0, end - buf); + REQUIRE (null_byte != NULL); + REQUIRE ((size_t) (null_byte - (char *) buf) + < sizeof (cie->augmentation)); strcpy (cie->augmentation, (char *) buf); - buf = (bfd_byte *) strchr ((char *) buf, '\0') + 1; + buf = (bfd_byte *) null_byte + 1; this_inf->u.cie.aug_str_len = buf - start - 1; ENSURE_NO_RELOCS (buf); if (buf[0] == 'e' && buf[1] == 'h')