Commit 5d1ae279 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman

staging: atomisp: silence an array overflow warning

Static checkers complain that we should check if "i" is in bounds
before we check if "var8[i]" is a NUL char.  This bug is harmless but
also easy to fix.
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2bbe9783
...@@ -669,7 +669,7 @@ int gmin_get_config_var(struct device *dev, const char *var, char *out, size_t * ...@@ -669,7 +669,7 @@ int gmin_get_config_var(struct device *dev, const char *var, char *out, size_t *
/* Our variable names are ASCII by construction, but EFI names /* Our variable names are ASCII by construction, but EFI names
* are wide chars. Convert and zero-pad. */ * are wide chars. Convert and zero-pad. */
memset(var16, 0, sizeof(var16)); memset(var16, 0, sizeof(var16));
for (i=0; var8[i] && i < sizeof(var8); i++) for (i = 0; i < sizeof(var8) && var8[i]; i++)
var16[i] = var8[i]; var16[i] = var8[i];
/* To avoid owerflows when calling the efivar API */ /* To avoid owerflows when calling the efivar API */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment