From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from osg.samsung.com ([64.30.133.232]:35562 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933124AbdKAU7z (ORCPT ); Wed, 1 Nov 2017 16:59:55 -0400 Date: Wed, 1 Nov 2017 18:59:48 -0200 From: Mauro Carvalho Chehab Cc: Linux Media Mailing List , Mauro Carvalho Chehab , Songjun Wu Subject: Re: [PATCH v2 01/26] media: atmel-isc: avoid returning a random value at isc_parse_dt() Message-ID: <20171101185948.4a5d91c6@vento.lan> In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit To: unlisted-recipients:; (no To-header on input)@bombadil.infradead.org Sender: linux-media-owner@vger.kernel.org List-ID: Em Wed, 1 Nov 2017 16:56:33 -0400 Mauro Carvalho Chehab escreveu: > As warned by smatch: > drivers/media/platform/atmel/atmel-isc.c:2097 isc_parse_dt() error: uninitialized symbol 'ret'. > > The problem here is that of_graph_get_next_endpoint() can > potentially return NULL on its first pass, with would make > it return a random value, as ret is not initialized. > > While here, use while(1) instead of for(; ;), as while is > the preferred syntax for such kind of loops. Sorry, please discard this e-mail... there's something wrong on my environment. git send-email is dying after the first e-mail: Died at /usr/libexec/git-core/git-send-email line 1350. I'll try to fix and re-send it. Thanks, Mauro