diff options
| author | Miaohe Lin <linmiaohe@huawei.com> | 2020-08-15 04:44:31 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-08-27 09:30:48 +0200 |
| commit | 72e6a4353a22ad996f6213d6ce4cf244a98b41d8 (patch) | |
| tree | 86b2a2a0618dc0b30346ba34866917730d80524d | |
| parent | 41ab1c624b215f9e0783bbad258e57126ef79749 (diff) | |
| download | linux-72e6a4353a22ad996f6213d6ce4cf244a98b41d8.tar.gz linux-72e6a4353a22ad996f6213d6ce4cf244a98b41d8.zip | |
net: Fix potential wrong skb->protocol in skb_vlan_untag()
[ Upstream commit 55eff0eb7460c3d50716ed9eccf22257b046ca92 ]
We may access the two bytes after vlan_hdr in vlan_set_encap_proto(). So
we should pull VLAN_HLEN + sizeof(unsigned short) in skb_vlan_untag() or
we may access the wrong data.
Fixes: 0d5501c1c828 ("net: Always untag vlan-tagged traffic on input.")
Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | net/core/skbuff.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/skbuff.c b/net/core/skbuff.c index 7e29590482ce..115f3fde314f 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.c @@ -5421,8 +5421,8 @@ struct sk_buff *skb_vlan_untag(struct sk_buff *skb) skb = skb_share_check(skb, GFP_ATOMIC); if (unlikely(!skb)) goto err_free; - - if (unlikely(!pskb_may_pull(skb, VLAN_HLEN))) + /* We may access the two bytes after vlan_hdr in vlan_set_encap_proto(). */ + if (unlikely(!pskb_may_pull(skb, VLAN_HLEN + sizeof(unsigned short)))) goto err_free; vhdr = (struct vlan_hdr *)skb->data; |