On 9/3/24 20:55, zhangjiao2 wrote:
From: zhang jiao zhangjiao2@cmss.chinamobile.com
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@cmss.chinamobile.com
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