2022-06-29 12:56:53

by lishengyu

[permalink] [raw]
Subject: [PATCH BlueZ] avdtp: fix possible minor problems

It should always be considered that if send_request fails, sep
should be removed from the list and the requested memory freed;
---
profiles/audio/avdtp.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/profiles/audio/avdtp.c b/profiles/audio/avdtp.c
index 0d2213ea2..10ef380d4 100644
--- a/profiles/audio/avdtp.c
+++ b/profiles/audio/avdtp.c
@@ -2765,8 +2765,11 @@ static gboolean avdtp_discover_resp(struct avdtp *session,

ret = send_request(session, TRUE, NULL, getcap_cmd,
&req, sizeof(req));
- if (ret < 0)
+ if (ret < 0) {
+ session->seps = g_slist_remove(session->seps, sep);
+ g_free(sep);
break;
+ }
getcap_pending = TRUE;
}

--
2.20.1




2022-06-29 14:04:35

by bluez.test.bot

[permalink] [raw]
Subject: RE: [BlueZ] avdtp: fix possible minor problems

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=655015

---Test result---

Test Summary:
CheckPatch PASS 1.19 seconds
GitLint PASS 0.87 seconds
Prep - Setup ELL PASS 44.14 seconds
Build - Prep PASS 0.54 seconds
Build - Configure PASS 8.63 seconds
Build - Make PASS 1434.45 seconds
Make Check PASS 11.55 seconds
Make Check w/Valgrind PASS 450.88 seconds
Make Distcheck PASS 240.18 seconds
Build w/ext ELL - Configure PASS 8.66 seconds
Build w/ext ELL - Make PASS 1419.52 seconds
Incremental Build with patchesPASS 0.00 seconds



---
Regards,
Linux Bluetooth

2022-06-30 20:40:59

by patchwork-bot+bluetooth

[permalink] [raw]
Subject: Re: [PATCH BlueZ] avdtp: fix possible minor problems

Hello:

This patch was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <[email protected]>:

On Wed, 29 Jun 2022 20:46:56 +0800 you wrote:
> It should always be considered that if send_request fails, sep
> should be removed from the list and the requested memory freed;
> ---
> profiles/audio/avdtp.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)

Here is the summary with links:
- [BlueZ] avdtp: fix possible minor problems
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=44658fccacda

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html