Fix leak of struct call_thread_fsm in call_function_by_hand_dummy.
authorPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Tue, 1 Jan 2019 19:54:52 +0000 (20:54 +0100)
committerPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Wed, 2 Jan 2019 21:10:39 +0000 (22:10 +0100)
commite3319240732bc9543af8ced9d80ec89d0b629b2e
tree32a5d194bf7a25ee10727823865140809a46c03a
parent5d36dfb949751663bcb814b3cc780ed8f47442b4
Fix leak of struct call_thread_fsm in call_function_by_hand_dummy.

When the call does not complete, the call_thread_fsm allocated
by new_call_thread_fsm is not cleaned up and deleted, which causes
the following leak e.g. in gdb.base/callfuncs.exp:

==29263== 560 bytes in 7 blocks are definitely lost in loss record 2,833 of 3,341
==29263==    at 0x4C2E0BC: calloc (vg_replace_malloc.c:762)
==29263==    by 0x405110: xcalloc (common-utils.c:84)
==29263==    by 0x4E67EB: xcnew<call_thread_fsm> (poison.h:122)
==29263==    by 0x4E67EB: new_call_thread_fsm (infcall.c:516)
==29263==    by 0x4E67EB: call_function_by_hand_dummy(value*, type*, gdb::array_view<value*>, void (*)(void*, int), void*) (infcall.c:1154)
==29263==    by 0x4E784E: call_function_by_hand(value*, type*, gdb::array_view<value*>) (infcall.c:693)
==29263==    by 0x496111: eval_call(expression*, noside, int, value**, char const*, type*) [clone .isra.5] (eval.c:835)

Fix the leak by similarly doing cleanup/destroy when restoring
previous state machine.

Tested on debian/amd64, natively and under valgrind.

2019-01-02  Philippe Waroquiers  <philippe.waroquiers@skynet.be>

* infcall.c (call_function_by_hand_dummy): cleanup/destroy sm
 in case of call that did not complete.
gdb/ChangeLog
gdb/infcall.c