On Fri, Jun 24 2016, Dan Carpenter wrote:
> Return -ENOMEM if kmalloc() fails.
>
> Fixes: 9353afbbfa7b ('usb: gadget: f_fs: buffer data from ‘oversized’ OUT 
> requests')
> Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
>

Acked-by: Michal Nazarewicz <min...@mina86.com>

> diff --git a/drivers/usb/gadget/function/f_fs.c 
> b/drivers/usb/gadget/function/f_fs.c
> index a91fcb0..5c8429f 100644
> --- a/drivers/usb/gadget/function/f_fs.c
> +++ b/drivers/usb/gadget/function/f_fs.c
> @@ -775,6 +775,8 @@ static ssize_t __ffs_epfile_read_data(struct ffs_epfile 
> *epfile,
>  
>       data_len -= ret;
>       buf = kmalloc(sizeof(*buf) + data_len, GFP_KERNEL);
> +     if (!buf)
> +             return -ENOMEM;
>       buf->length = data_len;
>       buf->data = buf->storage;
>       memcpy(buf->storage, data + ret, data_len);

-- 
Best regards
ミハウ “𝓶𝓲𝓷𝓪86” ナザレヴイツ
«If at first you don’t succeed, give up skydiving»
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to