diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2008-02-05 12:15:50 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-02-05 12:15:50 +0100 |
| commit | 8cf229437fd826c32a44546899412b1eb3e1db6f (patch) | |
| tree | 3afef8d1a3839b1ede06c738a0847f06bb3da3dd /net | |
| parent | [INET]: Fix accidentally broken inet(6)_hash_connect's port offset calculations. (diff) | |
| download | linux-8cf229437fd826c32a44546899412b1eb3e1db6f.tar.xz linux-8cf229437fd826c32a44546899412b1eb3e1db6f.zip | |
[ICMP]: Restore pskb_pull calls in receive function
Somewhere along the development of my ICMP relookup patch the header
length check went AWOL on the non-IPsec path. This patch restores the
check.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/ipv4/icmp.c | 3 | ||||
| -rw-r--r-- | net/ipv6/icmp.c | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/net/ipv4/icmp.c b/net/ipv4/icmp.c index a7321a82df6d..a13c074dac09 100644 --- a/net/ipv4/icmp.c +++ b/net/ipv4/icmp.c @@ -1015,7 +1015,8 @@ int icmp_rcv(struct sk_buff *skb) goto error; } - __skb_pull(skb, sizeof(*icmph)); + if (!pskb_pull(skb, sizeof(*icmph))) + goto error; icmph = icmp_hdr(skb); diff --git a/net/ipv6/icmp.c b/net/ipv6/icmp.c index cbb5b9cf84ad..121d517bf91c 100644 --- a/net/ipv6/icmp.c +++ b/net/ipv6/icmp.c @@ -683,7 +683,8 @@ static int icmpv6_rcv(struct sk_buff *skb) } } - __skb_pull(skb, sizeof(*hdr)); + if (!pskb_pull(skb, sizeof(*hdr))) + goto discard_it; hdr = icmp6_hdr(skb); |