On Wed, Oct 11, 2017 at 12:36:43PM +0200, Takashi Iwai wrote: > usx2y driver sets up URBs containing the fixed endpoints without > validation. This may end up with an oops-like kernel warning when > submitted. > > For avoiding it, this patch adds the calls of the new sanity-check > helper for URBs. > > Signed-off-by: Takashi Iwai <tiwai@xxxxxxx> > --- > sound/usb/usx2y/usbusx2y.c | 8 ++++++++ > sound/usb/usx2y/usbusx2yaudio.c | 3 +++ > 2 files changed, 11 insertions(+) > > diff --git a/sound/usb/usx2y/usbusx2y.c b/sound/usb/usx2y/usbusx2y.c > index 4569c0efac0a..55a631ccfa25 100644 > --- a/sound/usb/usx2y/usbusx2y.c > +++ b/sound/usb/usx2y/usbusx2y.c > @@ -244,6 +244,9 @@ static void i_usX2Y_In04Int(struct urb *urb) > usb_sndbulkpipe(usX2Y->dev, 0x04), &p4out->val.vol, > p4out->type == eLT_Light ? sizeof(struct us428_lights) : 5, > i_usX2Y_Out04Int, usX2Y); > + err = usb_urb_ep_type_check(usX2Y->AS04.urb[j]); > + if (err < 0) > + break; It doesn't make much sense to add this check to the completion handler, where, if you ever get here, you already know that endpoint check at first submission succeeded. > err = usb_submit_urb(usX2Y->AS04.urb[j], GFP_ATOMIC); > us428ctls->p4outSent = send; > break; Johan _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
