commit 93d44c6242faaade92f71b75978bab585fc1fa72
author David Srbecky <dsrbecky@google.com> Tue May 02 16:51:39 2023 +0100
committer David Srbecky <dsrbecky@google.com> Wed May 03 12:39:32 2023 +0000
tree 4a84be24f8053f5e4c35dcc559e358d42c9fe2f1
parent df9b65dd8f757de7ee734a01e76ce66e13d65fcc

create_minidebuginfo: Avoid overflow of ABS symbols.

Bug: 278156681
Change-Id: I441e73cf2c473202eb9f72dba7ab28722fee37ce

libelffile/elf/elf_debug_reader.h

diff --git a/libelffile/elf/elf_debug_reader.h b/libelffile/elf/elf_debug_reader.h
index 6e01989..fc7ad56 100644
--- a/libelffile/elf/elf_debug_reader.h
+++ b/libelffile/elf/elf_debug_reader.h
@@ -127,7 +127,9 @@
       CHECK_EQ(symtab->sh_entsize, sizeof(Elf_Sym));
       size_t count = symtab->sh_size / sizeof(Elf_Sym);
       for (const Elf_Sym& symbol : Read<Elf_Sym>(symtab->sh_offset, count)) {
-        if (ELF_ST_TYPE(symbol.st_info) == STT_FUNC && &sections_[symbol.st_shndx] == text) {
+        if (ELF_ST_TYPE(symbol.st_info) == STT_FUNC &&
+            symbol.st_shndx < sections_.size() &&  // Ignore ABS section.
+            &sections_[symbol.st_shndx] == text) {
           visit_sym(symbol, Read<char>(strtab->sh_offset + symbol.st_name));
         }
       }