[PATCH] monitor: Remove superfluous call to l2cap_frame_pull

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Removed superfluous call to l2cap_frame_pull.
---
 monitor/avctp.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/monitor/avctp.c b/monitor/avctp.c
index 64d4b58..98ffa22 100644
--- a/monitor/avctp.c
+++ b/monitor/avctp.c
@@ -546,16 +546,13 @@ static bool avrcp_list_player_attributes(struct l2cap_frame *frame,
 						uint8_t ctype, uint8_t len,
 						uint8_t indent)
 {
-	struct l2cap_frame avrcp_frame;
 	uint8_t num;
 	int i;
 
 	if (len == 0)
 		return true;
 
-	l2cap_frame_pull(&avrcp_frame, frame, 0);
-
-	if (!l2cap_frame_get_u8(&avrcp_frame, &num))
+	if (!l2cap_frame_get_u8(frame, &num))
 		return false;
 
 	print_field("%*cAttributeCount: 0x%02x", (indent - 8), ' ', num);
@@ -563,7 +560,7 @@ static bool avrcp_list_player_attributes(struct l2cap_frame *frame,
 	for (i = 0; num > 0; num--, i++) {
 		uint8_t attr;
 
-		if (!l2cap_frame_get_u8(&avrcp_frame, &attr))
+		if (!l2cap_frame_get_u8(frame, &attr))
 			return false;
 
 		print_field("%*cAttributeID: 0x%02x (%s)", (indent - 8), ' ',
-- 
1.9.1

--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux