diff options
| author |   Jason A. Donenfeld <Jason@zx2c4.com> | 2017-06-04 04:16:23 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-06-04 23:01:47 -0400 |
| commit | 3f29770723fe498a5c5f57c3a31a996ebdde03e1 (patch) | |
| tree | d93c21ae11e1ca396088bc931fc9a479b9a1b211 /net/ipv4/ah4.c | |
| parent | skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow (diff) | |
| download | linux-dev-3f29770723fe498a5c5f57c3a31a996ebdde03e1.tar.xz linux-dev-3f29770723fe498a5c5f57c3a31a996ebdde03e1.zip | |
ipsec: check return value of skb_to_sgvec always
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Cc: Steffen Klassert <steffen.klassert@secunet.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/ipv4/ah4.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/net/ipv4/ah4.c b/net/ipv4/ah4.c index 22377c8ff14b..e8f862358518 100644 --- a/net/ipv4/ah4.c +++ b/net/ipv4/ah4.c @@ -220,7 +220,9 @@ static int ah_output(struct xfrm_state *x, struct sk_buff *skb) ah->seq_no = htonl(XFRM_SKB_CB(skb)->seq.output.low); sg_init_table(sg, nfrags + sglists); - skb_to_sgvec_nomark(skb, sg, 0, skb->len); + err = skb_to_sgvec_nomark(skb, sg, 0, skb->len); + if (unlikely(err < 0)) + goto out_free; if (x->props.flags & XFRM_STATE_ESN) { /* Attach seqhi sg right after packet payload */ @@ -393,7 +395,9 @@ static int ah_input(struct xfrm_state *x, struct sk_buff *skb) skb_push(skb, ihl); sg_init_table(sg, nfrags + sglists); - skb_to_sgvec_nomark(skb, sg, 0, skb->len); + err = skb_to_sgvec_nomark(skb, sg, 0, skb->len); + if (unlikely(err < 0)) + goto out_free; if (x->props.flags & XFRM_STATE_ESN) { /* Attach seqhi sg right after packet payload */ |