Merge pull request #1443 from lplewa/coverity

lukaszstolarczuk · web-flow · commit e2bab4ab6f3d · 2025-07-16T14:12:10.000+02:00
fix coverity issue
diff --git a/test/ctl/ctl_unittest.cpp b/test/ctl/ctl_unittest.cpp
@@ -110,8 +110,8 @@ void ctl_helper(struct ctl *ctl_handler, const char *name, int *out, ...) {
     va_start(args, out);
     umf_result_t ret = ctl_query(ctl_handler, NULL, CTL_QUERY_PROGRAMMATIC,
                                  name, CTL_QUERY_READ, out, sizeof(*out), args);
-    ASSERT_EQ(ret, UMF_RESULT_SUCCESS);
     va_end(args);
+    ASSERT_EQ(ret, UMF_RESULT_SUCCESS);
 }
 
 TEST_F(test, ctl_debug_node_arg) {

Original file line number	Diff line number	Diff line change
`@@ -110,8 +110,8 @@ void ctl_helper(struct ctl ctl_handler, const char name, int *out, ...) {`
`110`	`110`	`va_start(args, out);`
`111`	`111`	`umf_result_t ret = ctl_query(ctl_handler, NULL, CTL_QUERY_PROGRAMMATIC,`
`112`	`112`	`name, CTL_QUERY_READ, out, sizeof(*out), args);`
`113`		`- ASSERT_EQ(ret, UMF_RESULT_SUCCESS);`
`114`	`113`	`va_end(args);`
	`114`	`+ ASSERT_EQ(ret, UMF_RESULT_SUCCESS);`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`TEST_F(test, ctl_debug_node_arg) {`