RE: [PATCH v4.14] usb: gadget: f_fs: Fix setting of device and driver data cross-references

From: Andrew Gabbasov
Date: Mon Jul 05 2021 - 06:24:55 EST


Hello Greg,

> -----Original Message-----
> From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Sent: Monday, July 05, 2021 10:08 AM
> To: Gabbasov, Andrew <Andrew_Gabbasov@xxxxxxxxxx>
> Cc: Macpaul Lin <macpaul.lin@xxxxxxxxxxxx>; Eugeniu Rosca <erosca@xxxxxxxxxxxxxx>; linux-usb@xxxxxxxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx; Felipe Balbi <balbi@xxxxxxxxxx>; Eugeniu Rosca
> <roscaeugeniu@xxxxxxxxx>; Eddie Hung <eddie.hung@xxxxxxxxxxxx>
> Subject: Re: [PATCH v4.14] usb: gadget: f_fs: Fix setting of device and driver data cross-references
>
> On Fri, Jul 02, 2021 at 01:49:57PM -0500, Andrew Gabbasov wrote:
> > Fixes: 4b187fceec3c ("usb: gadget: FunctionFS: add devices management code")
> > Fixes: 3262ad824307 ("usb: gadget: f_fs: Stop ffs_closed NULL pointer dereference")
> > Fixes: cdafb6d8b8da ("usb: gadget: f_fs: Fix use-after-free in ffs_free_inst")
> > Reported-by: Bhuvanesh Surachari <bhuvanesh_surachari@xxxxxxxxxx>
> > Tested-by: Eugeniu Rosca <erosca@xxxxxxxxxxxxxx>
> > Reviewed-by: Eugeniu Rosca <erosca@xxxxxxxxxxxxxx>
> > Signed-off-by: Andrew Gabbasov <andrew_gabbasov@xxxxxxxxxx>
> > Link: https://lore.kernel.org/r/20210603171507.22514-1-andrew_gabbasov@xxxxxxxxxx
> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > (cherry-picked from commit ecfbd7b9054bddb12cea07fda41bb3a79a7b0149)
>
> There is no such commit id in Linus's tree :(
>
> Please resubmit with the correct id.

This commit is not yet included to the mainline, it only exists in linux-next:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=ecfbd7b9054bddb12cea07fda41bb3a79a7b0149

Could you please advise if I need to somehow denote the linux-next repo in the "cherry picked from" line,
or just remove this line, or so far wait and re-submit the patch after the original commit is merged to Linus' tree?
BTW, I just noticed that the line contains incorrect "cherry-picked" instead of "cherry picked",
so I'll have to re-submit the patch anyway 😉

Thanks!

Best regards,
Andrew