Luiz Augusto von Dentz | 24 May 2012 15:22
Picon

[PATCH BlueZ 1/6] AVCTP: Fix setting wrong transaction id expected for responses

From: Luiz Augusto von Dentz <luiz.von.dentz@...>

The id were incremented after being set to the request so it is always
+1 of the actual transaction.
---
 audio/avctp.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/audio/avctp.c b/audio/avctp.c
index 1bc2a1d..5f9afa1 100644
--- a/audio/avctp.c
+++ b/audio/avctp.c
 <at>  <at>  -982,7 +982,7  <at>  <at>  int avctp_send_vendordep_req(struct avctp *session, uint8_t code,
 	struct avctp_rsp_handler *handler;
 	int err;

-	err = avctp_send(session, id++, AVCTP_COMMAND, code, subunit,
+	err = avctp_send(session, id, AVCTP_COMMAND, code, subunit,
 				AVC_OP_VENDORDEP, operands, operand_count);
 	if (err < 0)
 		return err;
 <at>  <at>  -994,6 +994,8  <at>  <at>  int avctp_send_vendordep_req(struct avctp *session, uint8_t code,

 	session->handlers = g_slist_prepend(session->handlers, handler);

+	id++;
+
 	return 0;
 }

--

-- 
1.7.7.6


Gmane