ipsec: Revert "IPSEC: remove double byte swap of IP addresses"
This reverts commit
Type: fix
Fixes:
@@ -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));
