[PATCH BlueZ] media: Fix memory leak

[PATCH BlueZ] media: Fix memory leak

[Luiz Augusto von Dentz]

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

This fixes the following memory leak as a result of not freeing
app->proxies:

160 (32 direct, 128 indirect) bytes in 1 blocks are definitely lost in
loss record 218 of 261
   at 0x484086F: malloc (vg_replace_malloc.c:381)
   by 0x1CF80E: btd_malloc (util.c:33)
   by 0x1CF24D: queue_new (queue.c:47)
   by 0x144DB9: create_app (media.c:2262)
   by 0x144DB9: register_app (media.c:2322)
   by 0x1CC148: process_message (object.c:246)
---
 profiles/audio/media.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/profiles/audio/media.c b/profiles/audio/media.c
index 521902ed8..edefedc90 100644
--- a/profiles/audio/media.c
+++ b/profiles/audio/media.c
@@ -1910,6 +1910,7 @@ static void app_free(void *data)
 {
 	struct media_app *app = data;
 
+	queue_destroy(app->proxies, NULL);
 	queue_destroy(app->endpoints, media_endpoint_remove);
 	queue_destroy(app->players, media_player_remove);
 
-- 
2.31.1

RE: [BlueZ] media: Fix memory leak

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 884 bytes --]

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=577661

---Test result---

Test Summary:
CheckPatch                    PASS      1.30 seconds
GitLint                       PASS      0.87 seconds
Prep - Setup ELL              PASS      49.54 seconds
Build - Prep                  PASS      0.46 seconds
Build - Configure             PASS      9.05 seconds
Build - Make                  PASS      204.90 seconds
Make Check                    PASS      9.80 seconds
Make Distcheck                PASS      244.96 seconds
Build w/ext ELL - Configure   PASS      9.09 seconds
Build w/ext ELL - Make        PASS      194.61 seconds



---
Regards,
Linux Bluetooth

Re: [BlueZ] media: Fix memory leak

[Luiz Augusto von Dentz]

Hi,

On Tue, Nov 9, 2021 at 2:37 PM <bluez.test.bot@gmail.com> wrote:
>
> This is automated email and please do not reply to this email!
>
> Dear submitter,
>
> Thank you for submitting the patches to the linux bluetooth mailing list.
> This is a CI test results with your patch series:
> PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=577661
>
> ---Test result---
>
> Test Summary:
> CheckPatch                    PASS      1.30 seconds
> GitLint                       PASS      0.87 seconds
> Prep - Setup ELL              PASS      49.54 seconds
> Build - Prep                  PASS      0.46 seconds
> Build - Configure             PASS      9.05 seconds
> Build - Make                  PASS      204.90 seconds
> Make Check                    PASS      9.80 seconds
> Make Distcheck                PASS      244.96 seconds
> Build w/ext ELL - Configure   PASS      9.09 seconds
> Build w/ext ELL - Make        PASS      194.61 seconds
>
>
>
> ---
> Regards,
> Linux Bluetooth

Pushed.

-- 
Luiz Augusto von Dentz