[PATCH v7 01/14] usb: chipidea: remove unneeded NULL check

Greg KH gregkh at linuxfoundation.org
Wed Jun 27 07:43:11 EST 2012


On Mon, Jun 25, 2012 at 02:56:18PM +0800, Richard Zhao wrote:
> From: Alexander Shishkin <alexander.shishkin at linux.intel.com>
> 
> As reported by Dan Carpenter, there is a NULL check in udc_start() that
> follows a dereference of the pointer that's being checked. However, at
> that point udc pointer shouldn't ever be NULL and if it is, the dereference
> should cause an oops.
> 
> Signed-off-by: Alexander Shishkin <alexander.shishkin at linux.intel.com>
> Reported-by: Dan Carpenter <dan.carpenter at oracle.com>
> Acked-by: Felipe Balbi <balbi at ti.com>
> ---
>  drivers/usb/chipidea/udc.c |    3 ---
>  1 file changed, 3 deletions(-)

First comment, you send a bunch of patches, yet you didn't sign off on
some of them, why?  If you are forwarding on patches, please sign off
showing you have reviewed them (so you can be on the path-of-blame if
there is a problem...)

There has been some discussion on some of these patches, so I'm guessing
we need a v8 set of this series at the least?

thanks,

greg k-h


More information about the devicetree-discuss mailing list