diff options
author | Szymon Janc <szymon.janc@tieto.com> | 2014-01-09 17:56:13 +0100 |
---|---|---|
committer | Johan Hedberg <johan.hedberg@intel.com> | 2014-01-09 21:11:07 +0200 |
commit | 843087b51a53d8eab7463082b9761e4c037d3802 (patch) | |
tree | eb5759e62d0f44731a50294e47f6ebaeee1ee322 /lib | |
parent | d51dce6e8129f818012585959337aaf58af6211a (diff) | |
download | bluez-843087b51a53d8eab7463082b9761e4c037d3802.tar.gz |
shared: Fix not calling destroy callback while clearing up handlers
read/write_watch_destroy is be called after io_set_read/write_handler
exits. This was resulting in io destroy callback being NULLem before
destroy was called resulting in io destroy not being called at all.
Fix this by assigning destroy callback only if setting non-NULL
callback. destroy_watch will clean it when called.
Fix following valgrind report:
168 (120 direct, 48 indirect) bytes in 1 blocks are definitely lost in
loss record 34 of 36
at 0x4C2C494: calloc (in /usr/lib/valgrind/
vgpreload_memcheck-amd64-linux.so)
by 0x409BAE: mgmt_new (mgmt.c:372)
by 0x409D05: mgmt_new_default (mgmt.c:469)
by 0x40E100: test_pre_setup (android-tester.c:371)
by 0x40AD84: start_tester (tester.c:586)
by 0x4E7C3B5: g_main_context_dispatch (in /lib/x86_64-linux-gnu/
libglib-2.0.so.0.3800.1)
by 0x4E7C707: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.3800.1)
by 0x4E7CB09: g_main_loop_run (in /lib/x86_64-linux-gnu/
libglib-2.0.so.0.3800.1)
by 0x40B76C: tester_run (tester.c:784)
by 0x4034F4: main (android-tester.c:2127)
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions