Re: [PATCH] usb: chipidea: udc: improve error handling on ep_queue and f_midi

From: Felipe Balbi
Date: Fri Sep 18 2015 - 13:01:19 EST


Hi,

On Fri, Sep 18, 2015 at 05:48:45PM +0100, eu@xxxxxxxxxxxxxxxxx wrote:
> From: "Felipe F. Tonello" <eu@xxxxxxxxxxxxxxxxx>
>
> _ep_queue() didn't check for errors when using add_td_to_list()
> which can fail if dma_pool_alloc fails, thus causing a kernel
> panic when lastnode->ptr is NULL.
>
> Also f_midi is not checking weather the is an error on usb_ep_queue
> request, ignoring potential problems, such as memory leaks.
>
> Signed-off-by: Felipe F. Tonello <eu@xxxxxxxxxxxxxxxxx>
> ---
> drivers/usb/chipidea/udc.c | 26 +++++++++++++++++++-------
> drivers/usb/gadget/function/f_midi.c | 11 +++++++++--

why are you patching f_midi.c in a chipidea patch ? Have a read at
Documentation/SubmittingPatches, Documentation/SubmitChecklist

--
balbi

Attachment: signature.asc
Description: Digital signature