From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756810Ab0JaWTD (ORCPT ); Sun, 31 Oct 2010 18:19:03 -0400 Received: from mail-yw0-f46.google.com ([209.85.213.46]:60856 "EHLO mail-yw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753157Ab0JaWTA (ORCPT ); Sun, 31 Oct 2010 18:19:00 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to :mime-version:content-type:content-transfer-encoding:message-id; b=GKkqCEMSUwdOb4+ViABWhsY5YbumgZpZJzYNxMFQuk8S4NpfRWJfXGqGsvMIipa7do TUczi1819hwMTn+uUBG/OA7Lc9LdP/3rAnNtJGZ+ctymv/MtCyT2S7xDwVoJ8Pn/vUcu F7L4unhSYN/ZrZnd05iOohFC7f5eAbzJhXGgA= From: Dmitry Torokhov To: Stefan Richter , Linus Torvalds Subject: Re: drivers/media/IR/ir-keytable.c::ir_getkeycode - 'retval' may be used uninitialized Date: Sun, 31 Oct 2010 15:18:42 -0700 User-Agent: KMail/1.13.5 (Linux/2.6.36+; KDE/4.4.5; x86_64; ; ) Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, mchehab@redhat.com References: In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201010311518.42998.dmitry.torokhov@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sunday, October 31, 2010 10:51:21 am Stefan Richter wrote: > Commit 9f470095068e "Input: media/IR - switch to using new keycode > interface" added the following build warning: > > drivers/media/IR/ir-keytable.c: In function 'ir_getkeycode': > drivers/media/IR/ir-keytable.c:363: warning: 'retval' may be used uninitialized in this function > > It is due to an actual bug but I don't know the fix. > The patch below should fix it. I wonder if Linus released -rc1 yet... -- Dmitry Input: ir-keytable - fix uninitialized variable warning From: Dmitry Torokhov We were forgetting to set up proper return value in success path causing ir_getkeycode() to fail intermittently: drivers/media/IR/ir-keytable.c: In function 'ir_getkeycode': drivers/media/IR/ir-keytable.c:363: warning: 'retval' may be used uninitialized in this function Reported-by: Stefan Richter Signed-off-by: Dmitry Torokhov --- drivers/media/IR/ir-keytable.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/media/IR/ir-keytable.c b/drivers/media/IR/ir-keytable.c index 9186b45..f7fafff 100644 --- a/drivers/media/IR/ir-keytable.c +++ b/drivers/media/IR/ir-keytable.c @@ -389,6 +389,8 @@ static int ir_getkeycode(struct input_dev *dev, ke->len = sizeof(entry->scancode); memcpy(ke->scancode, &entry->scancode, sizeof(entry->scancode)); + retval = 0; + out: spin_unlock_irqrestore(&rc_tab->lock, flags); return retval;