[PATCH BlueZ] monitor: Fix printing params for HCI Intel DDC command

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

 



From: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>

The offset of value for HCI Intel DDC command is incorrect.
HCI Intel DDC parameter has a combination of following structure:

struct intel_ddc {
	u8  len;
	u16 id;
	u8  *value;
}

The offset of value should be sum of len and id which is 3.
---
 monitor/intel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/monitor/intel.c b/monitor/intel.c
index ce624be..4cd1788 100644
--- a/monitor/intel.c
+++ b/monitor/intel.c
@@ -437,7 +437,7 @@ static void ddc_config_write_cmd(const void *data, uint8_t size)
 		uint16_t param_id = get_le16(data + 1);
 
 		print_field("Identifier: 0x%4.4x", param_id);
-		packet_hexdump(data + 2, param_len - 2);
+		packet_hexdump(data + 3, param_len - 2);
 
 		data += param_len + 1;
 		size -= param_len + 1;
-- 
2.7.4

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