diff options
| -rw-r--r-- | src/vnet/udp/udp_cli.c | 11 | ||||
| -rw-r--r-- | src/vnet/udp/udp_local.c | 16 |
2 files changed, 14 insertions, 13 deletions
diff --git a/src/vnet/udp/udp_cli.c b/src/vnet/udp/udp_cli.c index d60f0dde50b..1e723923c2f 100644 --- a/src/vnet/udp/udp_cli.c +++ b/src/vnet/udp/udp_cli.c @@ -211,16 +211,21 @@ static void table_format_udp_port_ (vlib_main_t *vm, udp_main_t *um, table_t *t, int *c, int port, int bind, int is_ip4) { - const udp_dst_port_info_t *pi = udp_get_dst_port_info (um, port, is_ip4); - if (!pi) + const udp_dst_port_info_t *pi; + + if (bind && !udp_is_valid_dst_port (port, is_ip4)) return; - if (bind && ~0 == pi->node_index) + + pi = udp_get_dst_port_info (um, port, is_ip4); + if (!pi) return; + table_format_cell (t, *c, 0, "%d", pi->dst_port); table_format_cell (t, *c, 1, is_ip4 ? "ip4" : "ip6"); table_format_cell (t, *c, 2, ~0 == pi->node_index ? "none" : "%U", format_vlib_node_name, vm, pi->node_index); table_format_cell (t, *c, 3, "%s", pi->name); + (*c)++; } diff --git a/src/vnet/udp/udp_local.c b/src/vnet/udp/udp_local.c index 41938b8f1f7..88378ae04bc 100644 --- a/src/vnet/udp/udp_local.c +++ b/src/vnet/udp/udp_local.c @@ -523,16 +523,12 @@ u8 udp_is_valid_dst_port (udp_dst_port_t dst_port, u8 is_ip4) { udp_main_t *um = &udp_main; - u16 *n; - - if (is_ip4) - n = sparse_vec_validate (um->next_by_dst_port4, - clib_host_to_net_u16 (dst_port)); - else - n = sparse_vec_validate (um->next_by_dst_port6, - clib_host_to_net_u16 (dst_port)); - - return (n[0] != SPARSE_VEC_INVALID_INDEX && n[0] != UDP_NO_NODE_SET); + u16 *next_by_dst_port = + is_ip4 ? um->next_by_dst_port4 : um->next_by_dst_port6; + uword index = + sparse_vec_index (next_by_dst_port, clib_host_to_net_u16 (dst_port)); + return (index != SPARSE_VEC_INVALID_INDEX && + vec_elt (next_by_dst_port, index) != UDP_NO_NODE_SET); } void |