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: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c b/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
index 292949273d00871bb91c5941dbe5f2208f18e2b7..add00932f6c9ac636f55b5b98d9e1565fbe10894 100644 (file)
--- a/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
+++ b/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
@@ -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
         * are wide chars.  Convert and zero-pad. */
        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];
 
        /* To avoid owerflows when calling the efivar API */