test/eventdev: fix producer core validity checks

[dpdk.git] / app / test-bbdev / test_bbdev.c

diff --git a/app/test-bbdev/test_bbdev.c b/app/test-bbdev/test_bbdev.c
index 5d325b6..137c74c 100644 (file)
--- a/app/test-bbdev/test_bbdev.c
+++ b/app/test-bbdev/test_bbdev.c
@@ -14,6 +14,8 @@
 #include <rte_bbdev.h>
 #include <rte_bbdev_op.h>
 #include <rte_bbdev_pmd.h>
+#include<string.h>
+#include <rte_string_fns.h>
 
 #include "main.h"
 
@@ -788,14 +790,14 @@ test_bbdev_driver_init(void)
 
        /* Initialize the maximum amount of devices */
        do {
-               sprintf(name_tmp, "%s%i", "name_", num_devs);
+               snprintf(name_tmp, sizeof(name_tmp), "%s%i", "name_", num_devs);
                dev2 = rte_bbdev_allocate(name_tmp);
                TEST_ASSERT(dev2 != NULL,
                                "Failed to initialize bbdev driver");
                ++num_devs;
        } while (num_devs < (RTE_BBDEV_MAX_DEVS - 1));
 
-       sprintf(name_tmp, "%s%i", "name_", num_devs);
+       snprintf(name_tmp, sizeof(name_tmp), "%s%i", "name_", num_devs);
        dev2 = rte_bbdev_allocate(name_tmp);
        TEST_ASSERT(dev2 == NULL, "Failed to initialize bbdev driver number %d "
                        "more drivers than RTE_BBDEV_MAX_DEVS: %d ", num_devs,
@@ -804,7 +806,7 @@ test_bbdev_driver_init(void)
        num_devs--;
 
        while (num_devs >= num_devs_tmp) {
-               sprintf(name_tmp, "%s%i", "name_", num_devs);
+               snprintf(name_tmp, sizeof(name_tmp), "%s%i", "name_", num_devs);
                dev2 = rte_bbdev_get_named_dev(name_tmp);
                TEST_ASSERT_SUCCESS(rte_bbdev_release(dev2),
                                "Failed to uninitialize bbdev driver %s ",
@@ -825,7 +827,7 @@ test_bbdev_driver_init(void)
        TEST_ASSERT_FAIL(rte_bbdev_release(NULL),
                        "Failed to uninitialize bbdev driver with NULL bbdev");
 
-       sprintf(name_tmp, "%s", "invalid_name");
+       strlcpy(name_tmp, "invalid_name", sizeof(name_tmp));
        dev2 = rte_bbdev_get_named_dev(name_tmp);
        TEST_ASSERT_FAIL(rte_bbdev_release(dev2),
                        "Failed to uninitialize bbdev driver with invalid name");
@@ -895,12 +897,12 @@ test_bbdev_callback(void)
                        "Failed to callback rgstr for RTE_BBDEV_EVENT_UNKNOWN");
 
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_UNKNOWN, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process "
                        "for RTE_BBDEV_EVENT_UNKNOWN ");
 
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_ERROR, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process: "
                        "event RTE_BBDEV_EVENT_ERROR was not registered ");
 
@@ -919,12 +921,12 @@ test_bbdev_callback(void)
 
        event_status = -1;
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_UNKNOWN, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process "
                        "for RTE_BBDEV_EVENT_UNKNOWN ");
 
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_ERROR, NULL);
-       TEST_ASSERT(event_status == 1,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_ERROR,
                        "Failed test for rte_bbdev_pmd_callback_process "
                        "for RTE_BBDEV_EVENT_ERROR ");
 
@@ -938,12 +940,12 @@ test_bbdev_callback(void)
 
        event_status = -1;
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_UNKNOWN, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process "
                        "for RTE_BBDEV_EVENT_UNKNOWN ");
 
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_ERROR, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process: "
                        "event RTE_BBDEV_EVENT_ERROR was unregistered ");
 
@@ -992,7 +994,7 @@ test_bbdev_callback(void)
                "for RTE_BBDEV_EVENT_ERROR ");
 
        rte_bbdev_pmd_callback_process(dev2, RTE_BBDEV_EVENT_ERROR, NULL);
-       TEST_ASSERT(event_status == 1,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_ERROR,
                "Failed test for rte_bbdev_pmd_callback_process in dev2 "
                "for RTE_BBDEV_EVENT_ERROR ");
 
@@ -1006,7 +1008,7 @@ test_bbdev_callback(void)
                        "in dev 2 ");
 
        rte_bbdev_pmd_callback_process(dev2, RTE_BBDEV_EVENT_UNKNOWN, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process in dev2"
                        " for RTE_BBDEV_EVENT_UNKNOWN ");
 
@@ -1026,7 +1028,7 @@ test_bbdev_callback(void)
                " for RTE_BBDEV_EVENT_UNKNOWN ");
 
        rte_bbdev_pmd_callback_process(dev1, RTE_BBDEV_EVENT_UNKNOWN, NULL);
-       TEST_ASSERT(event_status == 0,
+       TEST_ASSERT(event_status == (int) RTE_BBDEV_EVENT_UNKNOWN,
                        "Failed test for rte_bbdev_pmd_callback_process in dev2 "
                        "for RTE_BBDEV_EVENT_UNKNOWN ");