If the host interface name is not specified at creation, host_if_name
was wrongly set to a stack-allocated variable. Make sure it always
points to a heap allocated vector.
At deletion time, we must free all allocated vectors.
Type:fix
Change-Id: I17751f38e95097998d51225fdccbf3ce3c365593
Signed-off-by: Benoît Ganne <bganne@cisco.com>
vif->ifindex = if_nametoindex (ifr.ifr_ifrn.ifrn_name);
if (!args->host_if_name)
vif->ifindex = if_nametoindex (ifr.ifr_ifrn.ifrn_name);
if (!args->host_if_name)
- args->host_if_name = (u8 *) ifr.ifr_ifrn.ifrn_name;
+ args->host_if_name = format (0, "%s", ifr.ifr_ifrn.ifrn_name);
unsigned int offload = 0;
hdrsz = sizeof (struct virtio_net_hdr_v1);
unsigned int offload = 0;
hdrsz = sizeof (struct virtio_net_hdr_v1);
vec_free (vif->rxq_vrings);
vec_free (vif->txq_vrings);
vec_free (vif->rxq_vrings);
vec_free (vif->txq_vrings);
+ vec_free (vif->host_if_name);
+ vec_free (vif->net_ns);
+ vec_free (vif->host_bridge);
+
tm->tap_ids = clib_bitmap_set (tm->tap_ids, vif->id, 0);
clib_memset (vif, 0, sizeof (*vif));
pool_put (mm->interfaces, vif);
tm->tap_ids = clib_bitmap_set (tm->tap_ids, vif->id, 0);
clib_memset (vif, 0, sizeof (*vif));
pool_put (mm->interfaces, vif);