diff options
author | Ravi kumar Veeramally <ravikumar.veeramally@linux.intel.com> | 2014-06-17 14:19:27 +0300 |
---|---|---|
committer | Szymon Janc <szymon.janc@tieto.com> | 2014-06-17 14:27:31 +0200 |
commit | 7eeb611f56855592c0f07fa9ad62954989e972aa (patch) | |
tree | 3f5afd2f958e09acca296642af2e0ccba6437c0d /profiles/health | |
parent | b6c68a1da10a4950e2bf63a922c056135f7ea0f8 (diff) | |
download | bluez-7eeb611f56855592c0f07fa9ad62954989e972aa.tar.gz |
profiles/health/hdp: Fix memory leak in SDP record preparation
Sdp record preparation part is copied to android/health.c from
profiles/health/hdp_utils.c. Memory leak is noticed while testing. Memory
summay is from android daemon, but code snippet is same. It is already fixed
in android/health.c while submitting android related patches.
==12515== 286 (16 direct, 270 indirect) bytes in 1 blocks are definitely lost in loss record 158 of 165
==12515== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==12515== by 0x44AC45: sdp_list_append (sdp.c:1743)
==12515== by 0x4398F9: register_features (health.c:381)
==12515== by 0x4091CC: queue_foreach (queue.c:219)
==12515== by 0x43A31F: bt_health_mdep_cfg_data (health.c:398)
==12515== by 0x418B50: cmd_watch_cb (ipc.c:144)
==12515== by 0x4E7FCE4: g_main_context_dispatch (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515== by 0x4E80047: ??? (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515== by 0x4E80309: g_main_loop_run (in /lib/x86_64-linux-gnu/libglib-2.0.so.0.4000.0)
==12515== by 0x4044F2: main (main.c:538)
Diffstat (limited to 'profiles/health')
-rw-r--r-- | profiles/health/hdp_util.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/profiles/health/hdp_util.c b/profiles/health/hdp_util.c index 58770b51c..47c464e5a 100644 --- a/profiles/health/hdp_util.c +++ b/profiles/health/hdp_util.c @@ -652,6 +652,8 @@ static gboolean register_service_sup_features(GSList *app_list, return FALSE; } + sdp_list_free(sup_features, free_hdp_list); + return TRUE; } |