ipv6: fix ip6_tnl_parse_tlv_enc_lim()
authorEric Dumazet <edumazet@google.com>
Tue, 24 Jan 2017 00:43:06 +0000 (16:43 -0800)
committerWilly Tarreau <w@1wt.eu>
Wed, 7 Jun 2017 22:47:10 +0000 (00:47 +0200)
commit72bbf335e7aad09c88c50dbdd238f4faabd12174
tree14bf7085c4c9f2af297e29b68503438d8f2e9a2a
parent3bdb157105639fdcdff744432760c3f25c545678
ipv6: fix ip6_tnl_parse_tlv_enc_lim()

commit fbfa743a9d2a0ffa24251764f10afc13eb21e739 upstream.

This function suffers from multiple issues.

First one is that pskb_may_pull() may reallocate skb->head,
so the 'raw' pointer needs either to be reloaded or not used at all.

Second issue is that NEXTHDR_DEST handling does not validate
that the options are present in skb->data, so we might read
garbage or access non existent memory.

With help from Willem de Bruijn.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: Dmitry Vyukov <dvyukov@google.com>
Cc: Willem de Bruijn <willemb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Willy Tarreau <w@1wt.eu>
net/ipv6/ip6_tunnel.c