mirror of
https://github.com/torvalds/linux.git
synced 2026-11-07 06:17:02 +01:00
ALSA: caiaq: Allow zero-length packet handling for EP1
The recent fix to avoid OOB access for malformed packets added a
packet length check and returns immediately if it's too short.
However, this can be lead to a regression because the completion
handler doesn't resubmit the packet, while a zero-length packet itself
is valid, per se.
For addressing the potential regression above, change the handling for
a zero-length packet to just resubmit without processing the incoming
contents.
Fixes: aba30af07d ("ALSA: usb-audio: caiaq: validate EP1 reply lengths")
Link: https://patch.msgid.link/20261006134405.479603-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
parent
5cc1749b5f
commit
ccfe6ab672
|
|
@ -143,7 +143,7 @@ static void usb_ep1_command_reply_dispatch (struct urb* urb)
|
|||
return;
|
||||
}
|
||||
if (urb->actual_length < 1)
|
||||
return;
|
||||
goto resubmit;
|
||||
|
||||
payload_len = urb->actual_length - 1;
|
||||
|
||||
|
|
@ -193,6 +193,7 @@ static void usb_ep1_command_reply_dispatch (struct urb* urb)
|
|||
break;
|
||||
}
|
||||
|
||||
resubmit:
|
||||
cdev->ep1_in_urb->actual_length = 0;
|
||||
ret = usb_submit_urb(cdev->ep1_in_urb, GFP_ATOMIC);
|
||||
if (ret < 0)
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user