Re: [PATCH] staging: binder: cmd_name must be initialised

From: Dmitry Voytik
Date: Mon Sep 08 2014 - 08:43:22 EST


On Fri, Sep 5, 2014 at 10:48 PM, Greg Kroah-Hartman
<gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, Sep 05, 2014 at 08:02:24PM +0400, Dmitry Voytik wrote:
>> Variable cmd_name must be initialised as variable name is.
>
> Why? What bug does this fix? Have you tested this change? Does this
> cause binder to now act differently than before?

Hi Greg,

Thank you for your time.
The intent was to eliminate the GCC warning:
/scratch/kisskb/src/drivers/staging/android/binder.c: warning:
'cmd_name' may be used uninitialized in this function
[-Wuninitialized]: => 2275:5
But now after carefull review of the sorce code I've understood that
this is a false positive.
Sorry for bothering.

> thanks,
>
> greg k-h



--
Best Regards,
Dmitry Voytik.
voytikd@xxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/