Re: [PATCH USB 1/2] usb: raw-gadget: fix return value of ep read ioctls

From: Andrey Konovalov
Date: Fri Apr 24 2020 - 09:16:49 EST


On Fri, Apr 24, 2020 at 10:43 AM Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
>
> On Fri, Apr 24, 2020 at 03:09:58AM +0200, Andrey Konovalov wrote:
> > They must return the number of bytes transferred during the data stage.
> >
> > Signed-off-by: Andrey Konovalov <andreyknvl@xxxxxxxxxx>
>
> This was my mistake. Please add a Fixes tag.
>
> Fixes: 068fbff4f860 ("usb: raw-gadget: Fix copy_to/from_user() checks")
>
> I should have seen that bug... I thought I was being careful and I
> even singled out that part of the commit and mentioned it in the
> commit message but I messed up. Sorry.

No worries, the bug was actually present before your change, but in a
slightly different form. So FWIW we can also add:

Fixes: f2c2e717642c ("usb: gadget: add raw-gadget interface")

However raw-gadget is not present in any released kernel yet, so need
of backporting AFAIU.