ip_frag: fix build with gcc 5.1
authorBruce Richardson <bruce.richardson@intel.com>
Fri, 29 May 2015 14:34:14 +0000 (15:34 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 2 Jun 2015 16:24:28 +0000 (18:24 +0200)
On Fedora 22, with GCC 5.1, errors are reported due to array accesses
being potentially out of bounds. This commit fixes this by adding in an
extra bounds check to the loop counter.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
lib/librte_ip_frag/ip_frag_common.h

index 210f409..6b2acee 100644 (file)
@@ -90,7 +90,7 @@ static inline int
 ip_frag_key_is_empty(const struct ip_frag_key * key)
 {
        uint32_t i;
-       for (i = 0; i < key->key_len; i++)
+       for (i = 0; i < RTE_MIN(key->key_len, RTE_DIM(key->src_dst)); i++)
                if (key->src_dst[i] != 0)
                        return 0;
        return 1;