UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

EDAC, sb_edac: Fix signedness bugs in *_get_ha() functions 

A static checker gave the following warnings:

  drivers/edac/sb_edac.c:1030 ibridge_get_ha() warn: signedness bug returning '(-22)'
  drivers/edac/sb_edac.c:1037 knl_get_ha() warn: signedness bug returning '(-22)'

Both because the functions are declared to return a "u8", but try to
return -EINVAL for the error case.

Fix by returning 0xff (since the caller doesn't look at, or pass on, the
return value).

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Cc: linux-edac <linux-edac@vger.kernel.org>
Link: http://lkml.kernel.org/r/20180914201905.GA30946@agluck-desk
Signed-off-by: Borislav Petkov <bp@suse.de>
This commit is contained in:
Luck, Tony 2018-09-14 13:19:05 -07:00 committed by Borislav Petkov
parent 8489b17ce2
commit c968ed0859
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/edac/sb_edac.c
View File

@ -1027,14 +1027,14 @@ static u8 ibridge_get_ha(u8 bank)
case 9 ... 16: case 9 ... 16:
return (bank - 9) / 4; return (bank - 9) / 4;
default: default:
return -EINVAL; return 0xff;
} }
} }
/* Not used, but included for safety/symmetry */ /* Not used, but included for safety/symmetry */
static u8 knl_get_ha(u8 bank) static u8 knl_get_ha(u8 bank)
{ {
return -EINVAL; return 0xff;
} }
static u64 haswell_get_tolm(struct sbridge_pvt *pvt) static u64 haswell_get_tolm(struct sbridge_pvt *pvt)