leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

media: staging: atomisp: avoid a warning if 32 bits build

Browse Source 
Checking if a size_t value is bigger than ULONG_INT only makes
sense if building on 64 bits, as warned by:
	drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c:697 gmin_get_config_var() warn: impossible condition '(*out_len > (~0)) => (0-u32max > u32max)'

Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
This commit is contained in:
Mauro Carvalho Chehab 2018-03-26 15:06:00 -04:00
parent 8b1a2468cf
commit dc9f65cf9a
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
View File

@ -693,9 +693,11 @@ static int gmin_get_config_var(struct device *dev, const char *var,
for (i = 0; i < sizeof(var8) && var8[i]; i++) for (i = 0; i < sizeof(var8) && var8[i]; i++)
var16[i] = var8[i]; var16[i] = var8[i];
#ifdef CONFIG_64BIT
/* To avoid owerflows when calling the efivar API */ /* To avoid owerflows when calling the efivar API */
if (*out_len > ULONG_MAX) if (*out_len > ULONG_MAX)
return -EINVAL; return -EINVAL;
#endif
/* Not sure this API usage is kosher; efivar_entry_get()'s /* Not sure this API usage is kosher; efivar_entry_get()'s
* implementation simply uses VariableName and VendorGuid from * implementation simply uses VariableName and VendorGuid from