[PATCH] drivers: usb: host: fix prevent memory leak in interrupt message submission

Fabio Estevam festevam at gmail.com
Fri May 16 21:59:27 CEST 2025


On Fri, May 16, 2025 at 4:43 PM <ant.v.moryakov at gmail.com> wrote:

> @@ -1557,6 +1557,7 @@ static int _ehci_submit_int_msg(struct usb_device *dev, unsigned long pipe,
>                 debug("got wrong buffer back (%p instead of %p)\n",
>                       backbuffer, buffer);
>                 result = -EINVAL;
> +               goto err;

Was this supposed to be a v2? The error Tim pointed out is still here.

     ret = _ehci_destroy_int_queue(dev, queue);
> @@ -1565,6 +1566,10 @@ static int _ehci_submit_int_msg(struct usb_device *dev, unsigned long pipe,
>
>         /* everything worked out fine */
>         return result;
> +
> +err:
> +    _ehci_destroy_int_queue(dev, queue);  // Освобождаем очередь перед выходом

Please remove the comment.


More information about the U-Boot mailing list