rdma: fix non-NULL terminated C-string overflow

Type: fix

Change-Id: I584777825636347c116bd084e348858bcb1b7d75
Signed-off-by: Benoît Ganne <bganne@cisco.com>

diff --git a/src/plugins/rdma/device.c b/src/plugins/rdma/device.c
index 532f4f5..cb88849 100644 (file)
--- a/src/plugins/rdma/device.c
+++ b/src/plugins/rdma/device.c
@@ -324,7 +324,7 @@ rdma_async_event_init (rdma_device_t * rd)
   t.file_descriptor = rd->ctx->async_fd;
   t.error_function = rdma_async_event_error_ready;
   t.private_data = rd->dev_instance;
-  t.description = format (0, "%s async event", rd->name);
+  t.description = format (0, "%v async event", rd->name);
 
   rd->async_event_clib_file_index = clib_file_add (&file_main, &t);
   return 0;

diff --git a/src/plugins/rdma/format.c b/src/plugins/rdma/format.c
index 798b21d..7a97339 100644 (file)
--- a/src/plugins/rdma/format.c
+++ b/src/plugins/rdma/format.c
@@ -30,7 +30,7 @@ format_rdma_device_name (u8 * s, va_list * args)
   rdma_device_t *rd = vec_elt_at_index (rm->devices, i);
 
   if (rd->name)
-    return format (s, "%s", rd->name);
+    return format (s, "%v", rd->name);
 
   s = format (s, "rdma-%u", rd->dev_instance);
   return s;
@@ -59,7 +59,7 @@ format_rdma_device (u8 * s, va_list * args)
   rdma_device_t *rd = vec_elt_at_index (rm->devices, i);
   u32 indent = format_get_indent (s);
 
-  s = format (s, "netdev: %s\n", rd->linux_ifname);
+  s = format (s, "netdev: %v\n", rd->linux_ifname);
   s = format (s, "%Uflags: %U", format_white_space, indent,
              format_rdma_device_flags, rd);
   if (rd->error)