Re: [PATCH] ALSA: usb-audio: Fix a DMA to stack memory bug

From: Dan Carpenter
Date: Mon Dec 02 2024 - 13:55:06 EST


On Mon, Dec 02, 2024 at 04:05:01PM +0100, Benoît Sevens wrote:
> Hi Dan,
>
> On Mon, 2 Dec 2024 at 13:57, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> >
> > The usb_get_descriptor() function does DMA so we're not allowed
> > to use a stack buffer for that. Doing DMA to the stack is not portable
> > all architectures. Move the "new_device_descriptor" from being stored
> > on the stack and allocate it with kmalloc() instead.
> >
>
> Thanks for fixing this. It looks good to me.
>
> Note that the commit that is being fixed is already queued for
> backporting, so I don't know how this usually goes then.
>

It's fine. The stable scripts look for fixes to stable patches.

But I also CC'd stable because your commit is CC'd for stable. Even CC'ing
stable shouldn't be necessary here, maybe there is a rebase or something so the
Fixes tag gets broken or maybe something else goes wrong. CC'ing stable is just
an extra way to be careful.

regards,
dan carpenter