Re: [PATCH v2] usb: gadget: atmel: use calloc() to allocate endpoint list

[Mattijs Korpershoek <mkorpershoek@kernel.org>]

Hi Zixun,

Thank you for the patch.

On Sat, Jun 06, 2026 at 10:27, Zixun LI <admin@hifiphile.com> wrote:

> malloc() doesn't zero out memory, leaving ep->ep.enabled uninitiated,
> which could make this flag falsely true.
>
> In next usb_ep_enable() call since this flag is true, ep->ops->enable()
> will be skipped. Then usb_ep_queue() will fail on uninitialized endpoint.
>
> Fixes: 59310d1ecb9f ("usb: gadget: introduce 'enabled' flag in struct usb_ep")
> Signed-off-by: Zixun LI <admin@hifiphile.com>

Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org>

> ---
> On SAM9X60 I had a problem with unreliable usb_ether functionality,
> sometimes 'dhcp' command returns with error 'rx submit --> -108'.
> ---
> Changes in v2:
> - Add more details to commit message.
> - Link to v1: https://patch.msgid.link/20260521-udc_malloc-v1-1-b6118f809ada@hifiphile.com
> ---
>  drivers/usb/gadget/atmel_usba_udc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/gadget/atmel_usba_udc.c b/drivers/usb/gadget/atmel_usba_udc.c
> index f7a92ded6da..a5a2252dee9 100644
> --- a/drivers/usb/gadget/atmel_usba_udc.c
> +++ b/drivers/usb/gadget/atmel_usba_udc.c
> @@ -1219,7 +1219,7 @@ static struct usba_ep *usba_udc_pdata(struct usba_platform_data *pdata,
>  	struct usba_ep *eps;
>  	int i;
>  
> -	eps = malloc(sizeof(struct usba_ep) * pdata->num_ep);
> +	eps = calloc(pdata->num_ep, sizeof(struct usba_ep));
>  	if (!eps) {
>  		log_err("failed to alloc eps\n");
>  		return NULL;
>
> ---
> base-commit: 5732bd0f457b4c671e46574d64d4acb099c0f0a5
> change-id: 20260521-udc_malloc-eb8355d08afe
>
> Best regards,
> --  
> Zixun LI <admin@hifiphile.com>