Re: [bug report] mtd: rawnand: allocate dynamically ONFI parameters during detection

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Dan,

Dan Carpenter <dan.carpenter@oracle.com> wrote on Tue, 31 Jul 2018
21:26:54 +0300:

> Hello Miquel Raynal,
> 
> This is a semi-automatic email about new static checker warnings.
> 
> The patch 0ff393463e9c: "mtd: rawnand: allocate dynamically ONFI 
> parameters during detection" from Jul 25, 2018, leads to the 
> following Smatch complaint:
> 
>     drivers/mtd/nand/raw/nand_micron.c:93 micron_nand_onfi_init()
>      warn: variable dereferenced before check 'p->onfi' (see line 91)
> 
> drivers/mtd/nand/raw/nand_micron.c
>     90		struct nand_parameters *p = &chip->parameters;
>     91		struct nand_onfi_vendor_micron *micron = (void *)p->onfi->vendor;
>                                                                  ^^^^^^^^^^^^^^^
> The code dereferences "p->onfi"
> 
>     92	
>     93		if (p->onfi) {

That's right, I'll dereference p->onfi only in this block as it's not
used elsewhere.

Thanks,
Miquèl