From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> Checking (val & 0x7f) > 127 does not make any sense. --- profiles/audio/avrcp.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/profiles/audio/avrcp.c b/profiles/audio/avrcp.c index cd027c6..7731e88 100644 --- a/profiles/audio/avrcp.c +++ b/profiles/audio/avrcp.c @@ -1577,15 +1577,10 @@ static uint8_t avrcp_handle_set_absolute_volume(struct avrcp *session, { struct avrcp_player *player = session->controller->player; uint16_t len = ntohs(pdu->params_len); - uint8_t volume; if (len != 1) goto err; - volume = pdu->params[0] & 0x7F; - if (volume > 127) - goto err; - if (!player) goto err; -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html