Re: [bug report] pinctrl: sunxi: Deal with configless pins

[Maxime Ripard <maxime.ripard@free-electrons.com>]

[-- Attachment #1: Type: text/plain, Size: 1433 bytes --]

Hi Dan,

On Wed, Nov 16, 2016 at 03:31:24PM +0300, Dan Carpenter wrote:
> Hello Maxime Ripard,
> 
> The patch e11dee2e98f8: "pinctrl: sunxi: Deal with configless pins"
> from Oct 20, 2016, leads to the following static checker warning:
> 
> 	drivers/pinctrl/sunxi/pinctrl-sunxi.c:397 sunxi_pctrl_dt_node_to_map()
> 	warn: variable dereferenced before check 'map' (see line 323)
> 
> drivers/pinctrl/sunxi/pinctrl-sunxi.c
>    394          *num_maps = i;
>    395  
>    396          /*
>    397           * We know have the number of maps we need, we can resize our
>    398           * map array
>    399           */
>    400          *map = krealloc(*map, i * sizeof(struct pinctrl_map), GFP_KERNEL);
>    401          if (!map)
> 
> This should obviously be "if (*map) " here.

Erm, yes, totally.

> 
>    402                  return -ENOMEM;
>    403  
>    404          return 0;
>    405  
>    406  err_free_map:
>    407          kfree(map);
> 
> I would send a patch, but I wasn't sure about this.  Should this be
> kfree(*map);?

Yes, that one too.

> I don't have my cross function DB built for ARM so I
> don't know how this is called...  Maybe set *map to NULL as well before
> returning?

And that would be even better yes :)

Will you send a patch?

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]