[BlueZ,1/1] transport: Fix broadcast sink check

Commit Message

Vlad Pruteanu Sept. 5, 2024, 2:13 p.m. UTC

In transport_update_playing a check is performed to guard the path
which contains the TRANSPORT_STATE_BROADCASTING state. This branch
should be taken only when the device on which the code runs is a
broadcast sink. Thus, the UUID check is wrong, and this patch
updates it appropriately.
---
 profiles/audio/transport.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

[e-mail] Sept. 5, 2024, 4:15 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=887276

---Test result---

Test Summary:
CheckPatch                    PASS      0.46 seconds
GitLint                       PASS      0.32 seconds
BuildEll                      PASS      24.88 seconds
BluezMake                     PASS      1959.80 seconds
MakeCheck                     PASS      13.85 seconds
MakeDistcheck                 PASS      183.69 seconds
CheckValgrind                 PASS      258.77 seconds
CheckSmatch                   PASS      360.72 seconds
bluezmakeextell               PASS      123.36 seconds
IncrementalBuild              PASS      1754.22 seconds
ScanBuild                     PASS      1061.91 seconds



---
Regards,
Linux Bluetooth

Patch

diff --git a/profiles/audio/transport.c b/profiles/audio/transport.c
index 300145794..2c09d415e 100644
--- a/profiles/audio/transport.c
+++ b/profiles/audio/transport.c
@@ -1301,7 +1301,7 @@  static void transport_update_playing(struct media_transport *transport,
 
 	if (playing == FALSE) {
 		if (!strcmp(media_endpoint_get_uuid(transport->endpoint),
-						BCAA_SERVICE_UUID)) {
+						BAA_SERVICE_UUID)) {
 			if ((transport->state ==
 				TRANSPORT_STATE_BROADCASTING) ||
 				(transport->state == TRANSPORT_STATE_ACTIVE))