53 lines
1.8 KiB
Diff
53 lines
1.8 KiB
Diff
From 8f2c64de86bc3d7556121fe296dd679000283931 Mon Sep 17 00:00:00 2001
|
|
From: Alan Modra <amodra@gmail.com>
|
|
Date: Tue, 20 Dec 2022 23:47:03 +1030
|
|
Subject: [PATCH] PR29922, SHT_NOBITS section avoids section size sanity check
|
|
|
|
PR 29922
|
|
* dwarf2.c (find_debug_info): Ignore sections without
|
|
SEC_HAS_CONTENTS.
|
|
---
|
|
bfd/dwarf2.c | 12 +++++++++---
|
|
1 file changed, 9 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/bfd/dwarf2.c b/bfd/dwarf2.c
|
|
index 95f45708e9d..0cd8152ee6e 100644
|
|
--- a/bfd/dwarf2.c
|
|
+++ b/bfd/dwarf2.c
|
|
@@ -4831,16 +4831,19 @@ find_debug_info (bfd *abfd, const struct dwarf_debug_section *debug_sections,
|
|
{
|
|
look = debug_sections[debug_info].uncompressed_name;
|
|
msec = bfd_get_section_by_name (abfd, look);
|
|
- if (msec != NULL)
|
|
+ /* Testing SEC_HAS_CONTENTS is an anti-fuzzer measure. Of
|
|
+ course debug sections always have contents. */
|
|
+ if (msec != NULL && (msec->flags & SEC_HAS_CONTENTS) != 0)
|
|
return msec;
|
|
|
|
look = debug_sections[debug_info].compressed_name;
|
|
msec = bfd_get_section_by_name (abfd, look);
|
|
- if (msec != NULL)
|
|
+ if (msec != NULL && (msec->flags & SEC_HAS_CONTENTS) != 0)
|
|
return msec;
|
|
|
|
for (msec = abfd->sections; msec != NULL; msec = msec->next)
|
|
- if (startswith (msec->name, GNU_LINKONCE_INFO))
|
|
+ if ((msec->flags & SEC_HAS_CONTENTS) != 0
|
|
+ && startswith (msec->name, GNU_LINKONCE_INFO))
|
|
return msec;
|
|
|
|
return NULL;
|
|
@@ -4848,6 +4851,9 @@ find_debug_info (bfd *abfd, const struct dwarf_debug_section *debug_sections,
|
|
|
|
for (msec = after_sec->next; msec != NULL; msec = msec->next)
|
|
{
|
|
+ if ((msec->flags & SEC_HAS_CONTENTS) == 0)
|
|
+ continue;
|
|
+
|
|
look = debug_sections[debug_info].uncompressed_name;
|
|
if (strcmp (msec->name, look) == 0)
|
|
return msec;
|
|
--
|
|
2.39.3
|
|
|