[PATCH 21/21] staging: ks7010: init local variables when they are declared in ks7010_sdio_probe
Sergio Paracuellos
sergio.paracuellos at gmail.com
Mon Apr 23 13:19:49 UTC 2018
On Mon, Apr 23, 2018 at 2:45 PM, Dan Carpenter <dan.carpenter at oracle.com> wrote:
> On Thu, Apr 19, 2018 at 07:08:15AM +0200, Sergio Paracuellos wrote:
>> This commit change init point of two variables to forward them to
>> init time. This variables are just being assigned some lines after
>> and it is more clear to init them when the init value is known and
>> in this case this is known when they are declared.
>>
>> Signed-off-by: Sergio Paracuellos <sergio.paracuellos at gmail.com>
>> ---
>> drivers/staging/ks7010/ks7010_sdio.c | 7 ++-----
>> 1 file changed, 2 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c
>> index 11839af..18e2350 100644
>> --- a/drivers/staging/ks7010/ks7010_sdio.c
>> +++ b/drivers/staging/ks7010/ks7010_sdio.c
>> @@ -981,14 +981,11 @@ static void ks7010_private_init(struct ks_wlan_private *priv,
>> static int ks7010_sdio_probe(struct sdio_func *func,
>> const struct sdio_device_id *device)
>> {
>> - struct ks_wlan_private *priv;
>> + struct ks_wlan_private *priv = NULL;
>> + struct net_device *netdev = NULL;
>> struct ks_sdio_card *card;
>> - struct net_device *netdev;
>> int ret;
>>
>> - priv = NULL;
>> - netdev = NULL;
>
> It's better if we don't initialize these at all. There is a bug here
> and GCC finds it when these aren't initialized to bogus values:
>
> drivers/staging/ks7010/ks7010_sdio.c: In function ‘ks7010_sdio_probe’:
> drivers/staging/ks7010/ks7010_sdio.c:932:2: warning: ‘priv’ may be used uninitialized in this function [-Wmaybe-uninitialized]
I see... but I am not getting any warning in my tree making them
initialized. Am I missing something?
>
> regards,
> dan carpenter
Best regards,
Sergio Paracuellos
>
More information about the devel
mailing list