From dd2f12ba6ab952d9d66f4d9ba89ffde6309b1ff2 Mon Sep 17 00:00:00 2001 From: Benoît Ganne Date: Tue, 26 Apr 2022 18:44:19 +0200 Subject: ip: unformat_ip_address should no modify its argument on failure MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When failing to match an ip address, we should not reset the ip address that could have been initialized by a previous match. Type: fix Change-Id: I026766391eb3eb8230f75f66bf4b681e774741d9 Signed-off-by: Benoît Ganne --- src/vat/ip_types.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) (limited to 'src/vat') diff --git a/src/vat/ip_types.c b/src/vat/ip_types.c index f4dcc96febe..cdfcbc07de3 100644 --- a/src/vat/ip_types.c +++ b/src/vat/ip_types.c @@ -42,14 +42,15 @@ unformat_ip_address (unformat_input_t * input, va_list * args) { ip_address_t *a = va_arg (*args, ip_address_t *); - clib_memset (a, 0, sizeof (*a)); - if (unformat (input, "%U", unformat_ip4_address, &ip_addr_v4 (a))) - ip_addr_version (a) = AF_IP4; - else if (unformat_user (input, unformat_ip6_address, &ip_addr_v6 (a))) - ip_addr_version (a) = AF_IP6; - else - return 0; - return 1; + if (unformat_user (input, unformat_ip46_address, &ip_addr_46 (a), + IP46_TYPE_ANY)) + { + ip_addr_version (a) = + ip46_address_is_ip4 (&ip_addr_46 (a)) ? AF_IP4 : AF_IP6; + return 1; + } + + return 0; } u8 * -- cgit 1.2.3-korg