diff mbox series

[BlueZ] avdtp: fix possible minor problems

Message ID 20220629124656.10174-1-lishengyu@uniontech.com (mailing list archive)
State Accepted
Commit 44658fccacda3ade0ca2adbb2643b489671fe477
Headers show
Series [BlueZ] avdtp: fix possible minor problems | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/checkpatch success Checkpatch PASS
tedd_an/gitlint success Gitlint PASS
tedd_an/setupell success Setup ELL PASS
tedd_an/buildprep success Build Prep PASS
tedd_an/build success Build Configuration PASS
tedd_an/makecheck success Make Check PASS
tedd_an/makecheckvalgrind success Make Check PASS
tedd_an/makedistcheck success Make Distcheck PASS
tedd_an/build_extell success Build External ELL PASS
tedd_an/build_extell_make success Build Make with External ELL PASS

Commit Message

lishengyu June 29, 2022, 12:46 p.m. UTC
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(-)

Comments

bluez.test.bot@gmail.com June 29, 2022, 1:55 p.m. UTC | #1
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
patchwork-bot+bluetooth@kernel.org June 30, 2022, 8:40 p.m. UTC | #2
Hello:

This patch was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

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!
diff mbox series

Patch

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;
 	}