aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorangquan yu <angquan21@gmail.com>2023-11-28 21:36:15 -0600
committerShuah Khan <skhan@linuxfoundation.org>2023-11-30 14:32:41 -0700
commit9686e7f59b142095ac6ad0763312ec68fc34c51a (patch)
treed77a870b7aad39bdd1b09f5b3e700514c8300c1a
parentselftests/breakpoints: Fix format specifier in ksft_print_msg in step_after_suspend_test.c (diff)
downloadwireguard-linux-9686e7f59b142095ac6ad0763312ec68fc34c51a.tar.xz
wireguard-linux-9686e7f59b142095ac6ad0763312ec68fc34c51a.zip
selftests:x86: Fix Format String Warnings in lam.c
This commit addresses compiler warnings in lam.c related to the usage of non-literal format strings without format arguments in the 'run_test' function. Warnings fixed: - Resolved warnings indicating that 'ksft_test_result_skip' and 'ksft_test_result' were called with 't->msg' as a format string without accompanying format arguments. Changes made: - Modified the calls to 'ksft_test_result_skip' and 'ksft_test_result' to explicitly include a format specifier ("%s") for 't->msg'. - This ensures that the string is safely treated as a format argument, adhering to safer coding practices and resolving the compiler warnings. Signed-off-by: angquan yu <angquan21@gmail.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
-rw-r--r--tools/testing/selftests/x86/lam.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/x86/lam.c b/tools/testing/selftests/x86/lam.c
index 8f9b06d9ce03..215b8150b7cc 100644
--- a/tools/testing/selftests/x86/lam.c
+++ b/tools/testing/selftests/x86/lam.c
@@ -817,7 +817,7 @@ static void run_test(struct testcases *test, int count)
/* return 3 is not support LA57, the case should be skipped */
if (ret == 3) {
- ksft_test_result_skip(t->msg);
+ ksft_test_result_skip("%s", t->msg);
continue;
}
@@ -826,7 +826,7 @@ static void run_test(struct testcases *test, int count)
else
ret = !(t->expected);
- ksft_test_result(ret, t->msg);
+ ksft_test_result(ret, "%s", t->msg);
}
}