summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVladimir Isaev <visaev@netgate.com>2020-09-18 14:43:29 +0300
committerBeno�t Ganne <bganne@cisco.com>2020-09-18 14:57:04 +0000
commit84f3d9fba4e1715237a41f0978430193363187c7 (patch)
tree3aa189575c01c4a37c3ea26137f49e703f2edaec
parentb8b6fe4fcc9202b6811b90af8ac8f361dfff26bf (diff)
tap: do not use strlen on vector
sanitizer complains about strlen on hi->name in tap_dump_ifs. hi->name is a vector which is not null-terminated, so use vec_len. Type: fix Signed-off-by: Vladimir Isaev <visaev@netgate.com> Change-Id: Icdd5f65369bb51b0c4a9cd86c24899e6febd837c
-rw-r--r--src/vnet/devices/tap/tap.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/vnet/devices/tap/tap.c b/src/vnet/devices/tap/tap.c
index c265aa201ec..77c15ce0524 100644
--- a/src/vnet/devices/tap/tap.c
+++ b/src/vnet/devices/tap/tap.c
@@ -946,8 +946,7 @@ tap_dump_ifs (tap_interface_details_t ** out_tapids)
tapid->sw_if_index = vif->sw_if_index;
hi = vnet_get_hw_interface (vnm, vif->hw_if_index);
clib_memcpy(tapid->dev_name, hi->name,
- MIN (ARRAY_LEN (tapid->dev_name) - 1,
- strlen ((const char *) hi->name)));
+ MIN (ARRAY_LEN (tapid->dev_name) - 1, vec_len (hi->name)));
vring = vec_elt_at_index (vif->rxq_vrings, RX_QUEUE_ACCESS(0));
tapid->rx_ring_sz = vring->size;
vring = vec_elt_at_index (vif->txq_vrings, TX_QUEUE_ACCESS(0));