aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKingwel Xie <kingwel.xie@ericsson.com>2019-02-12 06:45:08 -0500
committerKingwel Xie <kingwel.xie@ericsson.com>2019-02-12 06:45:08 -0500
commit72b3bce0b6850f136748ba78bfa01b0192a0756d (patch)
treec5045388d1376de5ae272dc2201e2bee63c23b98
parent7ba8fe20970b501f06a6c5a6e3632913e2492acb (diff)
ipsec: cli bug fix
1. unformat_ip46_address must have ip-type specified 2. cannot unformat ip46_address_t with unformat_ip4_address Change-Id: I5f1eecfe71a808302428afb39f910ebf0c7cef71 Signed-off-by: Kingwel Xie <kingwel.xie@ericsson.com>
-rw-r--r--src/vnet/ipsec/ipsec_cli.c29
1 files changed, 23 insertions, 6 deletions
diff --git a/src/vnet/ipsec/ipsec_cli.c b/src/vnet/ipsec/ipsec_cli.c
index f66bf6d5aa1..52a30a428d0 100644
--- a/src/vnet/ipsec/ipsec_cli.c
+++ b/src/vnet/ipsec/ipsec_cli.c
@@ -118,14 +118,14 @@ ipsec_sa_add_del_command_fn (vlib_main_t * vm,
unformat_ipsec_integ_alg, &integ_alg))
;
else if (unformat (line_input, "tunnel-src %U",
- unformat_ip46_address, &tun_src))
+ unformat_ip46_address, &tun_src, IP46_TYPE_ANY))
{
flags |= IPSEC_SA_FLAG_IS_TUNNEL;
if (!ip46_address_is_ip4 (&tun_src))
flags |= IPSEC_SA_FLAG_IS_TUNNEL_V6;
}
else if (unformat (line_input, "tunnel-dst %U",
- unformat_ip46_address, &tun_dst))
+ unformat_ip46_address, &tun_dst, IP46_TYPE_ANY))
;
else if (unformat (line_input, "udp-encap"))
flags |= IPSEC_SA_FLAG_UDP_ENCAP;
@@ -615,6 +615,8 @@ create_ipsec_tunnel_command_fn (vlib_main_t * vm,
ipsec_add_del_tunnel_args_t a;
int rv;
u32 num_m_args = 0;
+ u8 ipv4_set = 0;
+ u8 ipv6_set = 0;
clib_error_t *error = NULL;
clib_memset (&a, 0, sizeof (a));
@@ -627,12 +629,21 @@ create_ipsec_tunnel_command_fn (vlib_main_t * vm,
while (unformat_check_input (line_input) != UNFORMAT_END_OF_INPUT)
{
if (unformat
- (line_input, "local-ip %U", unformat_ip4_address, &a.local_ip))
- num_m_args++;
+ (line_input, "local-ip %U", unformat_ip46_address, &a.local_ip,
+ IP46_TYPE_ANY))
+ {
+ ip46_address_is_ip4 (&a.local_ip) ? (ipv4_set = 1) : (ipv6_set = 1);
+ num_m_args++;
+ }
else
if (unformat
- (line_input, "remote-ip %U", unformat_ip4_address, &a.remote_ip))
- num_m_args++;
+ (line_input, "remote-ip %U", unformat_ip46_address, &a.remote_ip,
+ IP46_TYPE_ANY))
+ {
+ ip46_address_is_ip4 (&a.remote_ip) ? (ipv4_set = 1) : (ipv6_set =
+ 1);
+ num_m_args++;
+ }
else if (unformat (line_input, "local-spi %u", &a.local_spi))
num_m_args++;
else if (unformat (line_input, "remote-spi %u", &a.remote_spi))
@@ -663,6 +674,12 @@ create_ipsec_tunnel_command_fn (vlib_main_t * vm,
goto done;
}
+ if (ipv6_set)
+ return clib_error_return (0, "currently only IPv4 supported");
+
+ if (ipv4_set && ipv6_set)
+ return clib_error_return (0, "both IPv4 and IPv6 addresses specified");
+
rv = ipsec_add_del_tunnel_if (&a);
switch (rv)