redonkable
/
alistair23-linux
1
0
Fork 0
Code Releases Activity

ACPI, APEI, EINJ, cleanup 0 vs NULL confusion 

This function is returning pointers.  Sparse complains here:
drivers/acpi/apei/einj.c:262:32: warning:
	Using plain integer as NULL pointer

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Huang Ying <ying.huang@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
hifive-unleashed-5.1
Dan Carpenter 2012-01-20 10:57:14 +03:00 committed by Len Brown
parent 4c40aed869
commit 29924b9f8f
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/acpi/apei/einj.c
View File

@ -259,15 +259,15 @@ static void *einj_get_parameter_address(void)
v4param = ioremap(paddrv4, sizeof(*v4param));
if (!v4param)
return 0;
return NULL;
if (readq(&v4param->reserved1) || readq(&v4param->reserved2)) {
iounmap(v4param);
return 0;
return NULL;
}
return v4param;
}
return 0;
return NULL;
}
/* do sanity check to trigger table */