diff options
author | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-04-30 22:18:48 +0200 |
---|---|---|
committer | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-05-04 07:22:01 +0200 |
commit | 66452beb773568c7f60dbc6b7c70f671a8c9cf4b (patch) | |
tree | 9607e4db13f6abac22064fa6f0ee8b585940f661 /gdb/corelow.c | |
parent | bde09ab7026edf3d79122872b79c6a8f164ee0ff (diff) | |
download | binutils-66452beb773568c7f60dbc6b7c70f671a8c9cf4b.zip binutils-66452beb773568c7f60dbc6b7c70f671a8c9cf4b.tar.gz binutils-66452beb773568c7f60dbc6b7c70f671a8c9cf4b.tar.bz2 |
Fix leaks by clearing registers and frame caches.
Valgrind reports leaks such as the below in the tests:
gdb.threads/corethreads.exp
gdb.threads/gcore-thread.exp
gdb.ada/task_switch_in_core.exp
gdb.trace/tfile.exp
gdb.base/siginfo-thread.exp
==12701== 1,123 (72 direct, 1,051 indirect) bytes in 1 blocks are definitely lost in loss record 2,928 of 3,247
==12701== at 0x4C2C4CC: operator new(unsigned long) (vg_replace_malloc.c:344)
==12701== by 0x5CF771: get_thread_arch_aspace_regcache(ptid_t, gdbarch*, address_space*) (regcache.c:330)
==12701== by 0x5CF92A: get_thread_regcache (regcache.c:366)
==12701== by 0x5CF92A: get_current_regcache() (regcache.c:372)
==12701== by 0x4C7964: get_current_frame() (frame.c:1587)
==12701== by 0x4C7A3C: get_selected_frame(char const*) (frame.c:1651)
==12701== by 0x669EAD: print_thread_info_1(ui_out*, char const*, int, int, int) (thread.c:1151)
==12701== by 0x66A9A1: info_threads_command(char const*, int) (thread.c:1217)
==12701== by 0x40A878: cmd_func(cmd_list_element*, char const*, int) (cli-decode.c:1892)
...
Fix these leaks by clearing registers and frame caches.
This leak and fix is similar to the leak fixed by 799efbe8e01
Diffstat (limited to 'gdb/corelow.c')
-rw-r--r-- | gdb/corelow.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/gdb/corelow.c b/gdb/corelow.c index 3ce612d..9fcc2d2 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -535,6 +535,8 @@ core_target::detach (inferior *inf, int from_tty) 'this'. */ unpush_target (this); + /* Clear the register cache and the frame cache. */ + registers_changed (); reinit_frame_cache (); maybe_say_no_core_file_now (from_tty); } |