From 649f9360c471daa3210b41b4f261d2fe1a8bc1e0 Mon Sep 17 00:00:00 2001 From: Neale Ranns Date: Thu, 11 Jul 2019 09:31:19 +0000 Subject: ipsec: Revert "IPSEC: remove double byte swap of IP addresses" This reverts commit 9b208ced585d3b4620d6fde586cd047fe2027ecf. Type: fix Fixes: 9b208ced585d3b4620d6fde586cd047fe2027ecf Change-Id: I94a17039b4727bff0877423da5ba6cfceb188b17 Signed-off-by: Neale Ranns (cherry picked from commit d2029bc9c5947a8a676208bada9386e07ec16c97) --- src/vnet/ipsec/ipsec_input.c | 28 ++++++++++++++++++---------- 1 file changed, 18 insertions(+), 10 deletions(-) diff --git a/src/vnet/ipsec/ipsec_input.c b/src/vnet/ipsec/ipsec_input.c index 778570a41d4..5be72c3a161 100644 --- a/src/vnet/ipsec/ipsec_input.c +++ b/src/vnet/ipsec/ipsec_input.c @@ -86,25 +86,25 @@ ipsec_input_protect_policy_match (ipsec_spd_t * spd, u32 sa, u32 da, u32 spi) if (ipsec_sa_is_set_IS_TUNNEL (s)) { - if (da != s->tunnel_dst_addr.ip4.as_u32) + if (da != clib_net_to_host_u32 (s->tunnel_dst_addr.ip4.as_u32)) continue; - if (sa != s->tunnel_src_addr.ip4.as_u32) + if (sa != clib_net_to_host_u32 (s->tunnel_src_addr.ip4.as_u32)) continue; return p; } - if (da < p->laddr.start.ip4.as_u32) + if (da < clib_net_to_host_u32 (p->laddr.start.ip4.as_u32)) continue; - if (da > p->laddr.stop.ip4.as_u32) + if (da > clib_net_to_host_u32 (p->laddr.stop.ip4.as_u32)) continue; - if (sa < p->raddr.start.ip4.as_u32) + if (sa < clib_net_to_host_u32 (p->raddr.start.ip4.as_u32)) continue; - if (sa > p->raddr.stop.ip4.as_u32) + if (sa > clib_net_to_host_u32 (p->raddr.stop.ip4.as_u32)) continue; return p; @@ -233,8 +233,12 @@ VLIB_NODE_FN (ipsec4_input_node) (vlib_main_t * vm, /* FIXME TODO missing check whether there is enough data inside * IP/UDP to contain ESP header & stuff ? */ p0 = ipsec_input_protect_policy_match (spd0, - ip0->src_address.as_u32, - ip0->dst_address.as_u32, + clib_net_to_host_u32 + (ip0->src_address. + as_u32), + clib_net_to_host_u32 + (ip0->dst_address. + as_u32), clib_net_to_host_u32 (esp0->spi)); @@ -279,8 +283,12 @@ VLIB_NODE_FN (ipsec4_input_node) (vlib_main_t * vm, { ah0 = (ah_header_t *) ((u8 *) ip0 + ip4_header_bytes (ip0)); p0 = ipsec_input_protect_policy_match (spd0, - ip0->src_address.as_u32, - ip0->dst_address.as_u32, + clib_net_to_host_u32 + (ip0->src_address. + as_u32), + clib_net_to_host_u32 + (ip0->dst_address. + as_u32), clib_net_to_host_u32 (ah0->spi)); -- cgit 1.2.3-korg