diff options
| author | Eric Dumazet <edumazet@google.com> | 2017-01-24 01:43:06 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-01-24 20:53:24 +0100 |
| commit | fbfa743a9d2a0ffa24251764f10afc13eb21e739 (patch) | |
| tree | 391e471dde0dc5afd2f24ab0410d4a2a8761180b /net/caif | |
| parent | ip6_tunnel: must reload ipv6h in ip6ip6_tnl_xmit() (diff) | |
| download | linux-fbfa743a9d2a0ffa24251764f10afc13eb21e739.tar.xz linux-fbfa743a9d2a0ffa24251764f10afc13eb21e739.zip | |
ipv6: fix ip6_tnl_parse_tlv_enc_lim()
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>
Diffstat (limited to 'net/caif')
0 files changed, 0 insertions, 0 deletions