From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.linuxfoundation.org ([140.211.169.12]:47029 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753758Ab3KYRZJ (ORCPT ); Mon, 25 Nov 2013 12:25:09 -0500 Date: Mon, 25 Nov 2013 09:25:08 -0800 From: Greg Kroah-Hartman To: Michal Nazarewicz Cc: Dan Carpenter , devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Hans Verkuil , linux-media@vger.kernel.org, Mauro Carvalho Chehab Subject: Re: [PATCHv2] staging: go7007: fix use of uninitialised pointer Message-ID: <20131125172508.GA3874@kroah.com> References: <1384108677-23476-1-git-send-email-mpn@google.com> <20131110185210.GA9633@kroah.com> <87fvr480o9.fsf@mina86.com> <20131110210647.GA5302@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: linux-media-owner@vger.kernel.org List-ID: On Mon, Nov 11, 2013 at 12:46:24PM +0100, Michal Nazarewicz wrote: > go variable is initialised only after the switch case so it cannot be > dereferenced prior to that happening. > > Signed-off-by: Michal Nazarewicz > --- > drivers/staging/media/go7007/go7007-usb.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > On Sun, Nov 10 2013, Dan Carpenter wrote: > > There are 3 other uses before "go" gets initialized. > > Argh... Other occurrences of the letters “GO” deceived my eyes. Sorry > about that and thanks. This is no longer needed, as I revertd the patch that caused the original problems, sorry. greg k-h