On 9/3/24 20:55, zhangjiao2 wrote:
From: zhang jiao <zhangjiao2@xxxxxxxxxxxxxxxxxxxx>
Free string allocated by asprintf().
How did you find this problem? Include the details in
the change log - The tool and output from the tool.
Signed-off-by: zhang jiao <zhangjiao2@xxxxxxxxxxxxxxxxxxxx>
---
tools/testing/selftests/futex/functional/futex_requeue_pi.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/testing/selftests/futex/functional/futex_requeue_pi.c b/tools/testing/selftests/futex/functional/futex_requeue_pi.c
index 215c6cb539b4..fb2dab46087f 100644
--- a/tools/testing/selftests/futex/functional/futex_requeue_pi.c
+++ b/tools/testing/selftests/futex/functional/futex_requeue_pi.c
@@ -416,5 +416,8 @@ int main(int argc, char *argv[])
ret = unit_test(broadcast, locked, owner, timeout_ns);
print_result(test_name, ret);
+ if (strlen(test_name) > strlen(TEST_NAME))
+ free(test_name);
Why not set a flag to determine if test_name needs freeing
instead of calling strlen() twice?
+
return ret;
}
thanks,
-- Shuah
