usb: isp1760: check for null return from kzalloc
authorColin Ian King <colin.king@canonical.com>
Tue, 2 Jun 2015 18:05:13 +0000 (19:05 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Jun 2015 21:25:04 +0000 (14:25 -0700)
commite3e64f3f99d9e70973fb1ae7d02825fe2bb9eef2
treed03763df93c8b163fe86e97baaf62d2f6f3d5158
parent15bf722e6f6c0b884521a0363204532e849deb7f
usb: isp1760: check for null return from kzalloc

isp1760_ep_alloc_request allocates a structure with kzalloc without checking
for NULL and then returns a pointer to one of the structure fields. As the
field happens to be the first in the structure the caller can properly check
for NULL, but this is risky if the structure layout is changed later. Add an
explicit NULL check for the kzalloc return value

Detected with smatch static analysis:

drivers/usb/isp1760/isp1760-udc.c:816 isp1760_ep_alloc_request()
  error: potential null dereference 'req'.  (kzalloc returns null)

[ thanks to Laurent Pinchart for improved commit message ]

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/isp1760/isp1760-udc.c