Diff between 45aa22f0f3d672c66b9777317c12c4196a9efd86 and 3b205298c3de316c9ca959f8f02de7e3890d00ba

Changed Files

File Additions Deletions Status
android/avrcp.c +13 -8 modified

Full Patch

diff --git a/android/avrcp.c b/android/avrcp.c
index 504ce2b..cb46dcc 100644
--- a/android/avrcp.c
+++ b/android/avrcp.c
@@ -171,25 +171,28 @@ static void handle_get_player_values_text(const void *buf, uint16_t len)
 			HAL_OP_AVRCP_GET_PLAYER_VALUES_TEXT, HAL_STATUS_FAILED);
 }
 
-static void write_element_text(uint8_t id, uint8_t text_len, uint8_t *text,
-						uint8_t *pdu, size_t *len)
+static size_t write_element_text(uint8_t id, uint8_t text_len, uint8_t *text,
+						uint8_t *pdu)
 {
 	uint16_t charset = 106;
+	size_t len = 0;
 
 	bt_put_be32(id, pdu);
 	pdu += 4;
-	*len += 4;
+	len += 4;
 
 	bt_put_be16(charset, pdu);
 	pdu += 2;
-	*len += 2;
+	len += 2;
 
 	bt_put_be16(text_len, pdu);
 	pdu += 2;
-	*len += 2;
+	len += 2;
 
 	memcpy(pdu, text, text_len);
-	*len += text_len;
+	len += text_len;
+
+	return len;
 }
 
 static void write_element_attrs(uint8_t *ptr, uint8_t number, uint8_t *pdu,
@@ -203,11 +206,13 @@ static void write_element_attrs(uint8_t *ptr, uint8_t number, uint8_t *pdu,
 
 	for (i = 0; i < number; i++) {
 		struct hal_avrcp_player_setting_text *text = (void *) ptr;
+		size_t ret;
 
-		write_element_text(text->id, text->len, text->text, pdu, len);
+		ret = write_element_text(text->id, text->len, text->text, pdu);
 
 		ptr += sizeof(*text) + text->len;
-		pdu += *len;
+		pdu += ret;
+		*len += ret;
 	}
 }