Commit 09b3048b authored by Oliver Neukum's avatar Oliver Neukum Committed by Sasha Levin

ACM gadget: fix endianness in notifications

[ Upstream commit cdd7928d ]

The gadget code exports the bitfield for serial status changes
over the wire in its internal endianness. The fix is to convert
to little endian before sending it over the wire.
Signed-off-by: default avatarOliver Neukum <oneukum@suse.com>
Tested-by: default avatar家瑋 <momo1208@gmail.com>
CC: stable@vger.kernel.org
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarSasha Levin <alexander.levin@verizon.com>
parent 861b5c18
...@@ -540,13 +540,15 @@ static int acm_notify_serial_state(struct f_acm *acm) ...@@ -540,13 +540,15 @@ static int acm_notify_serial_state(struct f_acm *acm)
{ {
struct usb_composite_dev *cdev = acm->port.func.config->cdev; struct usb_composite_dev *cdev = acm->port.func.config->cdev;
int status; int status;
__le16 serial_state;
spin_lock(&acm->lock); spin_lock(&acm->lock);
if (acm->notify_req) { if (acm->notify_req) {
dev_dbg(&cdev->gadget->dev, "acm ttyGS%d serial state %04x\n", dev_dbg(&cdev->gadget->dev, "acm ttyGS%d serial state %04x\n",
acm->port_num, acm->serial_state); acm->port_num, acm->serial_state);
serial_state = cpu_to_le16(acm->serial_state);
status = acm_cdc_notify(acm, USB_CDC_NOTIFY_SERIAL_STATE, status = acm_cdc_notify(acm, USB_CDC_NOTIFY_SERIAL_STATE,
0, &acm->serial_state, sizeof(acm->serial_state)); 0, &serial_state, sizeof(acm->serial_state));
} else { } else {
acm->pending = true; acm->pending = true;
status = 0; status = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment