diff options
| author | Alan Modra <amodra@gmail.com> | 2019-09-02 09:54:55 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2019-09-02 10:08:17 +0930 |
| commit | 22b31fea07587034636251ccb81041e4e8e0bd93 (patch) | |
| tree | 5485ca7806b23b07d21790842f821eadae1f8067 /bfd/dwarf2.c | |
| parent | 579e12e52d00929ed3cf9e801fbc97d4594a1ecf (diff) | |
| download | gdb-22b31fea07587034636251ccb81041e4e8e0bd93.zip gdb-22b31fea07587034636251ccb81041e4e8e0bd93.tar.gz gdb-22b31fea07587034636251ccb81041e4e8e0bd93.tar.bz2 | |
Memory leak in dwarf2.c
This fixes a small leak of debug_filename. bfd_openr copies the file
name since git commit 1be5090bca.
PR 11983
* dwarf2.c (_bfd_dwarf2_slurp_debug_info): Free debug_filename
on success. Tidy.
Diffstat (limited to 'bfd/dwarf2.c')
| -rw-r--r-- | bfd/dwarf2.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/bfd/dwarf2.c b/bfd/dwarf2.c index a8beec4..88638b9 100644 --- a/bfd/dwarf2.c +++ b/bfd/dwarf2.c @@ -4385,18 +4385,20 @@ _bfd_dwarf2_slurp_debug_info (bfd *abfd, bfd *debug_bfd, fail more quickly. */ return FALSE; + debug_bfd = bfd_openr (debug_filename, NULL); + free (debug_filename); + if (debug_bfd == NULL) + /* FIXME: Should we report our failure to follow the debuglink ? */ + return FALSE; + /* Set BFD_DECOMPRESS to decompress debug sections. */ - if ((debug_bfd = bfd_openr (debug_filename, NULL)) == NULL - || !(debug_bfd->flags |= BFD_DECOMPRESS, - bfd_check_format (debug_bfd, bfd_object)) + debug_bfd->flags |= BFD_DECOMPRESS; + if (!bfd_check_format (debug_bfd, bfd_object) || (msec = find_debug_info (debug_bfd, debug_sections, NULL)) == NULL || !bfd_generic_link_read_symbols (debug_bfd)) { - if (debug_bfd) - bfd_close (debug_bfd); - /* FIXME: Should we report our failure to follow the debuglink ? */ - free (debug_filename); + bfd_close (debug_bfd); return FALSE; } |