test/devargs: fix memory leak
authorXueming Li <xuemingl@nvidia.com>
Sat, 23 Oct 2021 12:17:55 +0000 (20:17 +0800)
committerDavid Marchand <david.marchand@redhat.com>
Mon, 25 Oct 2021 07:42:38 +0000 (09:42 +0200)
In layer argument test function, kvargs are parsed and checked without
free. This patch calls rte_kvargs_free() function to avoid memory leak.

Coverity issue: 373631
Fixes: a4975cd20dca ("test: add devargs test cases")

Signed-off-by: Xueming Li <xuemingl@nvidia.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
app/test/test_devargs.c

index 1903671..1662128 100644 (file)
@@ -43,8 +43,10 @@ test_args(const char *devargs, const char *layer, const char *args, const int n)
        if ((int)kvlist->count != n) {
                printf("rte_devargs_parse(%s) %s_str: %s kv number %u, not %d\n",
                       devargs, layer, args, kvlist->count, n);
+               rte_kvargs_free(kvlist);
                return -1;
        }
+       rte_kvargs_free(kvlist);
        return 0;
 }