mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-11-03 09:01:54 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

bluez5: backend-native: Only send one error message

Browse source 
When rfcomm_hfp_ag() returns false, an "ERROR" reply is sent.
When testing if the SLC is configured, 2 "ERROR" replies are sent, which
should not be done.
This commit is contained in:
Frédéric Danis 2022-09-15 16:19:26 +02:00
parent 9755cb9083
commit 75ae86bf13
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
spa/plugins/bluez5/backend-native.c
View file

@ -809,7 +809,7 @@ static bool rfcomm_hfp_ag(struct rfcomm *rfcomm, char* buf)
} else if (!rfcomm->slc_configured) { } else if (!rfcomm->slc_configured) {
spa_log_warn(backend->log, "RFCOMM receive command before SLC completed: %s", buf); spa_log_warn(backend->log, "RFCOMM receive command before SLC completed: %s", buf);
rfcomm_send_reply(rfcomm, "ERROR"); rfcomm_send_reply(rfcomm, "ERROR");
return false; return true;
} else if (sscanf(buf, "AT+BCS=%u", &selected_codec) == 1) { } else if (sscanf(buf, "AT+BCS=%u", &selected_codec) == 1) {
/* parse BCS(=Bluetooth Codec Selection) reply */ /* parse BCS(=Bluetooth Codec Selection) reply */
bool was_switching_codec = rfcomm->hfp_ag_switching_codec && (rfcomm->device != NULL); bool was_switching_codec = rfcomm->hfp_ag_switching_codec && (rfcomm->device != NULL);