test/event: replace sprintf with snprintf
authorPallantla Poornima <pallantlax.poornima@intel.com>
Wed, 6 Feb 2019 10:43:42 +0000 (10:43 +0000)
committerJerin Jacob <jerinj@marvell.com>
Tue, 2 Apr 2019 01:32:56 +0000 (03:32 +0200)
sprintf function is not secure as it doesn't check the length of string.
More secure function snprintf is used.

Fixes: 2a9c83ae3b ("test/eventdev: add multi-ports test")
Cc: stable@dpdk.org
Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
app/test/test_event_eth_rx_adapter.c

index 1d3be82..38f5c03 100644 (file)
@@ -479,7 +479,8 @@ adapter_multi_eth_add_del(void)
        /* add the max port for rx_adapter */
        port_index = rte_eth_dev_count_total();
        for (; port_index < RTE_MAX_ETHPORTS; port_index += 1) {
-               sprintf(driver_name, "%s%u", "net_null", drv_id);
+               snprintf(driver_name, sizeof(driver_name), "%s%u", "net_null",
+                               drv_id);
                err = rte_vdev_init(driver_name, NULL);
                TEST_ASSERT(err == 0, "Failed driver %s got %d",
                driver_name, err);