test/devargs: fix memory leak

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>

diff --git a/app/test/test_devargs.c b/app/test/test_devargs.c
index 19036716bf7efbfa9b02e3cd967d034daacf2547..16621285d2dc585ed73087641dfdf3e95ccee117 100644 (file)
--- a/app/test/test_devargs.c
+++ b/app/test/test_devargs.c
@@ -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;
 }