usb: class: hid: Fix fault due to unaligned access

SavinayDharmappa · nashif · commit 0576f93ff844 · 2018-11-21T12:25:22.000-05:00
patch fix fault due to unaligned access while setting hid report size on xtensa platform. Fixes #11266 Signed-off-by: Savinay Dharmappa <savinay.dharmappa@intel.com>
diff --git a/subsys/usb/class/hid/core.c b/subsys/usb/class/hid/core.c
@@ -91,8 +91,8 @@ static struct hid_device_info {
 
 static void usb_set_hid_report_size(u16_t report_desc_size)
 {
-	hid_cfg.if0_hid.subdesc[0].wDescriptorLength =
-		sys_cpu_to_le16(report_desc_size);
+	UNALIGNED_PUT(sys_cpu_to_le16(report_desc_size),
+		&(hid_cfg.if0_hid.subdesc[0].wDescriptorLength));
 }
 
 static void hid_status_cb(enum usb_dc_status_code status, const u8_t *param)

Original file line number	Diff line number	Diff line change
`@@ -91,8 +91,8 @@ static struct hid_device_info {`
`91`	`91`
`92`	`92`	`static void usb_set_hid_report_size(u16_t report_desc_size)`
`93`	`93`	`{`
`94`		`- hid_cfg.if0_hid.subdesc[0].wDescriptorLength =`
`95`		`- sys_cpu_to_le16(report_desc_size);`
	`94`	`+ UNALIGNED_PUT(sys_cpu_to_le16(report_desc_size),`
	`95`	`+ &(hid_cfg.if0_hid.subdesc[0].wDescriptorLength));`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`static void hid_status_cb(enum usb_dc_status_code status, const u8_t *param)`