ipv6: pointer math error in ip6_tnl_parse_tlv_enc_lim()
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 1 Feb 2017 08:46:32 +0000 (11:46 +0300)
committerWilly Tarreau <w@1wt.eu>
Wed, 7 Jun 2017 22:47:10 +0000 (00:47 +0200)
commit 63117f09c768be05a0bf465911297dc76394f686 upstream.

Casting is a high precedence operation but "off" and "i" are in terms of
bytes so we need to have some parenthesis here.

Fixes: fbfa743a9d2a ("ipv6: fix ip6_tnl_parse_tlv_enc_lim()")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Willy Tarreau <w@1wt.eu>
net/ipv6/ip6_tunnel.c

index 33bf1c1f8ee7e56078b3e29ef246522e38585e9d..61a89951e7fed27bd0d192c818a78b934fad74b4 100644 (file)
@@ -435,7 +435,7 @@ __u16 ip6_tnl_parse_tlv_enc_lim(struct sk_buff *skb, __u8 *raw)
                                if (i + sizeof(*tel) > optlen)
                                        break;
 
-                               tel = (struct ipv6_tlv_tnl_enc_lim *) skb->data + off + i;
+                               tel = (struct ipv6_tlv_tnl_enc_lim *)(skb->data + off + i);
                                /* return index of option if found and valid */
                                if (tel->type == IPV6_TLV_TNL_ENCAP_LIMIT &&
                                    tel->length == 1)