Thread (9 messages) flat view 9 messages, 3 authors, 2011-11-04
STALE5400d

[PATCH] Fix check for PDU size

From: Lucas De Marchi <hidden>
Date: 2011-10-26 13:02:25
Subsystem: the rest · Maintainer: Linus Torvalds

Company ID became part of the avrcp header so its size is already
accounted in the AVRCP_HEADER_LENGTH define.
---
 audio/avrcp.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/audio/avrcp.c b/audio/avrcp.c
index f06afbf..879e959 100644
--- a/audio/avrcp.c
+++ b/audio/avrcp.c
@@ -1053,7 +1053,7 @@ static size_t handle_vendordep_pdu(struct avctp *session, uint8_t transaction,
 	pdu->packet_type = 0;
 	pdu->rsvd = 0;
 
-	if (operand_count + 3 < AVRCP_HEADER_LENGTH) {
+	if (operand_count < AVRCP_HEADER_LENGTH) {
 		pdu->params[0] = E_INVALID_COMMAND;
 		goto err_metadata;
 	}
-- 
1.7.7.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help