[bug report] mtd: parsers: Add Qcom SMEM parser

Manivannan Sadhasivam manivannan.sadhasivam at linaro.org
Fri Jan 8 08:28:28 EST 2021


Hi Dan,

On Fri, Jan 08, 2021 at 03:52:41PM +0300, Dan Carpenter wrote:
> Hello Manivannan Sadhasivam,
> 
> The patch 803eb124e1a6: "mtd: parsers: Add Qcom SMEM parser" from Jan
> 4, 2021, leads to the following static checker warning:
> 
> 	drivers/mtd/parsers/qcomsmempart.c:109 parse_qcomsmem_part()
> 	warn: passing zero to 'PTR_ERR'
> 
> drivers/mtd/parsers/qcomsmempart.c
>     68          pr_debug("Parsing partition table info from SMEM\n");
>     69          ptable = qcom_smem_get(SMEM_APPS, SMEM_AARM_PARTITION_TABLE, &len);
>     70          if (IS_ERR(ptable)) {
> 
> Can "ptable" be NULL here?
> 

No it won't. It will return valid error ptr in failure cases.

Thanks,
Mani



More information about the linux-mtd mailing list