This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: staging: atomisp: avoid a warning if 32 bits build
Author:  Mauro Carvalho Chehab <mche...@s-opensource.com>
Date:    Mon Mar 26 15:06:00 2018 -0400

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.ai...@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

 .../staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c    | 2 ++
 1 file changed, 2 insertions(+)

---

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 be0c5e11e86b..3283c1b05d6a 100644
--- a/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
+++ b/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
@@ -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++)
                var16[i] = var8[i];
 
+#ifdef CONFIG_64BIT
        /* To avoid owerflows when calling the efivar API */
        if (*out_len > ULONG_MAX)
                return -EINVAL;
+#endif
 
        /* Not sure this API usage is kosher; efivar_entry_get()'s
         * implementation simply uses VariableName and VendorGuid from

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to