From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH v7 01/14] usb: chipidea: remove unneeded NULL check Date: Tue, 26 Jun 2012 14:43:11 -0700 Message-ID: <20120626214311.GA4878@kroah.com> References: <1340607391-16380-1-git-send-email-richard.zhao@freescale.com> <1340607391-16380-2-git-send-email-richard.zhao@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1340607391-16380-2-git-send-email-richard.zhao-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Richard Zhao Cc: linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, B29397-KZfg59tc24xl57MIdRCFDg@public.gmane.org, B20596-KZfg59tc24xl57MIdRCFDg@public.gmane.org, marex-ynQEQJNshbs@public.gmane.org, shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, alexander.shishkin-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org, dong.aisheng-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org, linuxzsc-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, balbi-l0cyMroinI0@public.gmane.org List-Id: devicetree@vger.kernel.org On Mon, Jun 25, 2012 at 02:56:18PM +0800, Richard Zhao wrote: > From: Alexander Shishkin > > 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 > Reported-by: Dan Carpenter > Acked-by: Felipe Balbi > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html