diff options
author | Alan Modra <amodra@gmail.com> | 2023-06-14 14:24:50 +0930 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2023-06-14 14:24:50 +0930 |
commit | 6f860418d556d4e5492b3da9e1a52e4b85a85f3e (patch) | |
tree | 7d8223edf575df45e809a9d2068b518ac67086d6 /ld/NEWS | |
parent | 48375c36dc59e3c0e6b9bf38f17f4692bddb578f (diff) | |
download | gdb-6f860418d556d4e5492b3da9e1a52e4b85a85f3e.zip gdb-6f860418d556d4e5492b3da9e1a52e4b85a85f3e.tar.gz gdb-6f860418d556d4e5492b3da9e1a52e4b85a85f3e.tar.bz2 |
asprintf memory leaks
A number of backends want to return bfd_reloc_dangerous messaqes from
relocation special_function, and construct the message using asprintf.
Such messages are not freed anywhere, leading to small memory leaks
inside libbfd. To limit the leaks, I'd implemented a static buffer in
the ppc backends that was freed before use in asprintf output. This
patch extends that scheme to other backends using a shared static
buffer and goes further in freeing the buffer on any bfd_close.
The patch also fixes a few other cases where asprintf output was not
freed after use.
bfd/
* bfd.c (_input_error_msg): Make global and rename to..
(_bfd_error_buf): ..this.
(bfd_asprintf): New function.
(bfd_errmsg): Use bfd_asprintf.
* opncls.c (bfd_close_all_done): Free _buf_error_buf.
* elf32-arm.c (find_thumb_glue, find_arm_glue): Use bfd_asprintf.
* elf32-nios2.c (nios2_elf32_relocate_section): Likewise.
* elf32-ppc.c (ppc_elf_unhandled_reloc): Likewise.
* elf64-ppc.c (ppc64_elf_unhandled_reloc): Likewise.
* elfnn-riscv.c (riscv_resolve_pcrel_lo_relocs): Likewise.
(riscv_elf_relocate_section): Likewise.
* libbfd.h: Regenerate.
gas/
* read.c (read_end): Free current_name and current_label.
(do_s_func): Likewise on error path. strdup label.
ld/
* pe-dll.c (make_head, make_tail, make_one),
(make_singleton_name_thunk, make_import_fixup_entry),
(make_runtime_pseudo_reloc),
(pe_create_runtime_relocator_reference: Free oname after use.
Diffstat (limited to 'ld/NEWS')
0 files changed, 0 insertions, 0 deletions