include/linux/byteorder/generic.h: fix index variables
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Sun, 23 May 2021 20:49:57 +0000 (22:49 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 26 Nov 2021 15:19:19 +0000 (16:19 +0100)
In cpu_to_be32_array() and be32_to_cpu_array() the length of the array is
given by variable len of type size_t. An index variable of type int is used
to iterate over the array. This is bound to fail for len > INT_MAX and
lets GCC add instructions for sign extension.

Correct the type of the index variable.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Link: https://lore.kernel.org/r/20210523204958.64575-1-xypron.glpk@gmx.de
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
include/linux/byteorder/generic.h

index 4b13e0a3e15b2207cf6030d2ee8b2ecc7aef785c..c9a4c96c994308868e775d83e53469227d1f9b82 100644 (file)
@@ -190,7 +190,7 @@ static inline void be64_add_cpu(__be64 *var, u64 val)
 
 static inline void cpu_to_be32_array(__be32 *dst, const u32 *src, size_t len)
 {
-       int i;
+       size_t i;
 
        for (i = 0; i < len; i++)
                dst[i] = cpu_to_be32(src[i]);
@@ -198,7 +198,7 @@ static inline void cpu_to_be32_array(__be32 *dst, const u32 *src, size_t len)
 
 static inline void be32_to_cpu_array(u32 *dst, const __be32 *src, size_t len)
 {
-       int i;
+       size_t i;
 
        for (i = 0; i < len; i++)
                dst[i] = be32_to_cpu(src[i]);