| Message ID | f3c5eaa5-d1a3-cc7a-1bd8-20fa2c961770@strongswan.org |
|---|---|
| State | Awaiting Upstream, archived |
| Delegated to: | David Miller |
| Headers | show |
On Tue, Nov 29, 2016 at 05:05:25PM +0100, Tobias Brunner wrote: > When handling inbound packets, the two halves of the sequence number > stored on the skb are already in network order. > > Fixes: 000ae7b2690e ("esp6: Switch to new AEAD interface") > Signed-off-by: Tobias Brunner <tobias@strongswan.org> Acked-by: Herbert Xu <herbert@gondor.apana.org.au> Thanks.
On Wed, Nov 30, 2016 at 05:58:38PM +0800, Herbert Xu wrote: > On Tue, Nov 29, 2016 at 05:05:25PM +0100, Tobias Brunner wrote: > > When handling inbound packets, the two halves of the sequence number > > stored on the skb are already in network order. > > > > Fixes: 000ae7b2690e ("esp6: Switch to new AEAD interface") > > Signed-off-by: Tobias Brunner <tobias@strongswan.org> > > Acked-by: Herbert Xu <herbert@gondor.apana.org.au> Also applied to the ipsec tree, thanks a lot everyone!
diff --git a/net/ipv6/esp6.c b/net/ipv6/esp6.c index 060a60b2f8a6..111ba55fd512 100644 --- a/net/ipv6/esp6.c +++ b/net/ipv6/esp6.c @@ -418,7 +418,7 @@ static int esp6_input(struct xfrm_state *x, struct sk_buff *skb) esph = (void *)skb_push(skb, 4); *seqhi = esph->spi; esph->spi = esph->seq_no; - esph->seq_no = htonl(XFRM_SKB_CB(skb)->seq.input.hi); + esph->seq_no = XFRM_SKB_CB(skb)->seq.input.hi; aead_request_set_callback(req, 0, esp_input_done_esn, skb); }
When handling inbound packets, the two halves of the sequence number stored on the skb are already in network order. Fixes: 000ae7b2690e ("esp6: Switch to new AEAD interface") Signed-off-by: Tobias Brunner <tobias@strongswan.org> --- net/ipv6/esp6.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)