avrcp: Ignore MSB of Absolute Volume parameter

According to the Section 6.13.1 of the AVRCP 1.5 specification document,
the most significant bit of the Absolute Volume parameter is reserved
for future additions, so it should be ignored when parsing parameters of
the SetAbsoluteVolume command. This is also necessary to pass PTS test
TC_TG_VLH_BI_02_C.

diff --git a/profiles/audio/avrcp.c b/profiles/audio/avrcp.c
index 32062db..4965b0c 100644 (file)
--- a/profiles/audio/avrcp.c
+++ b/profiles/audio/avrcp.c
@@ -1399,11 +1399,13 @@ static uint8_t avrcp_handle_set_absolute_volume(struct avrcp *session,
 {
        struct avrcp_player *player = session->player;
        uint16_t len = ntohs(pdu->params_len);
+       uint8_t volume;
 
        if (len != 1)
                goto err;
 
-       if (pdu->params[0] > 127)
+       volume = pdu->params[0] & 0x7F;
+       if (volume > 127)
                goto err;
 
        if (!player)