From: Matt Ranostay <matt.ranostay@konsulko.com>
Date: Sat, 20 May 2017 20:46:12 +0000 (-0700)
Subject: binding: bluetooth: fix memory leak in avrcp controls
X-Git-Tag: dab_3.99.1^0
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F43%2F9443%2F1;p=apps%2Fsettings.git

binding: bluetooth: fix memory leak in avrcp controls

avrcp data pointers need to be freed when btd_device
structure is destroyed

Change-Id: I39b534665d9e7b74ff20443668b5fee395446aa9
Bug-AGL: SPEC-596
Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
---

diff --git a/binding-bluetooth/bluetooth-manager.c b/binding-bluetooth/bluetooth-manager.c
index 35c770f..1f18f8b 100644
--- a/binding-bluetooth/bluetooth-manager.c
+++ b/binding-bluetooth/bluetooth-manager.c
@@ -157,6 +157,24 @@ void device_free(struct btd_device* device)
         device->name = NULL;
     }
 
+    if (device->avrcp_title) {
+        D_PRINTF("avrcp_title:%s\n",device->avrcp_title);
+        g_free(device->avrcp_title);
+        device->avrcp_title = NULL;
+    }
+
+    if (device->avrcp_artist) {
+        D_PRINTF("avrcp_artist:%s\n",device->avrcp_artist);
+        g_free(device->avrcp_artist);
+        device->avrcp_artist = NULL;
+    }
+
+    if (device->avrcp_status) {
+        D_PRINTF("avrcp_status:%s\n",device->avrcp_status);
+        g_free(device->avrcp_status);
+        device->avrcp_status = NULL;
+    }
+
     g_free(device);
 }