raw/skeleton: add missing check after setting attribute
authorMin Hu (Connor) <humin29@huawei.com>
Thu, 22 Apr 2021 06:21:46 +0000 (14:21 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 12 May 2021 14:27:33 +0000 (16:27 +0200)
This patch adds return value check for setting an attribute.

Fixes: 88a81bcecb7b ("raw/skeleton: remove compile-time constant for device id")
Cc: stable@dpdk.org
Signed-off-by: Min Hu (Connor) <humin29@huawei.com>
drivers/raw/skeleton/skeleton_rawdev_test.c

index 1405df0..484468e 100644 (file)
@@ -295,6 +295,7 @@ test_rawdev_attr_set_get(void)
        dummy_value = &set_value;
        *dummy_value = 200;
        ret = rte_rawdev_set_attr(test_dev_id, "Test2", (uintptr_t)dummy_value);
+       RTE_TEST_ASSERT(!ret, "Unable to set an attribute (Test2)");
 
        /* Check if attributes have been set */
        ret = rte_rawdev_get_attr(test_dev_id, "Test1", &ret_value);