| Message ID | 20210917192614.24862-1-skhan@linuxfoundation.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 48514a22333099b93c33ea1c7c97dad3f7a611ce |
| Headers | show |
| Series | selftests: net: af_unix: Fix incorrect args in test result msg | expand |
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Fri, 17 Sep 2021 13:26:14 -0600 you wrote: > Fix the args to fprintf(). Splitting the message ends up passing > incorrect arg for "sigurg %d" and an extra arg overall. The test > result message ends up incorrect. > > test_unix_oob.c: In function ‘main’: > test_unix_oob.c:274:43: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘char *’ [-Wformat=] > 274 | fprintf(stderr, "Test 3 failed, sigurg %d len %d OOB %c ", > | ~^ > | | > | int > | %s > 275 | "atmark %d\n", signal_recvd, len, oob, atmark); > | ~~~~~~~~~~~~~ > | | > | char * > test_unix_oob.c:274:19: warning: too many arguments for format [-Wformat-extra-args] > 274 | fprintf(stderr, "Test 3 failed, sigurg %d len %d OOB %c ", > > [...] Here is the summary with links: - selftests: net: af_unix: Fix incorrect args in test result msg https://git.kernel.org/netdev/net/c/48514a223330 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/tools/testing/selftests/net/af_unix/test_unix_oob.c b/tools/testing/selftests/net/af_unix/test_unix_oob.c index 0f3e3763f4f8..3dece8b29253 100644 --- a/tools/testing/selftests/net/af_unix/test_unix_oob.c +++ b/tools/testing/selftests/net/af_unix/test_unix_oob.c @@ -271,8 +271,9 @@ main(int argc, char **argv) read_oob(pfd, &oob); if (!signal_recvd || len != 127 || oob != '%' || atmark != 1) { - fprintf(stderr, "Test 3 failed, sigurg %d len %d OOB %c ", - "atmark %d\n", signal_recvd, len, oob, atmark); + fprintf(stderr, + "Test 3 failed, sigurg %d len %d OOB %c atmark %d\n", + signal_recvd, len, oob, atmark); die(1); }
Fix the args to fprintf(). Splitting the message ends up passing incorrect arg for "sigurg %d" and an extra arg overall. The test result message ends up incorrect. test_unix_oob.c: In function ‘main’: test_unix_oob.c:274:43: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘char *’ [-Wformat=] 274 | fprintf(stderr, "Test 3 failed, sigurg %d len %d OOB %c ", | ~^ | | | int | %s 275 | "atmark %d\n", signal_recvd, len, oob, atmark); | ~~~~~~~~~~~~~ | | | char * test_unix_oob.c:274:19: warning: too many arguments for format [-Wformat-extra-args] 274 | fprintf(stderr, "Test 3 failed, sigurg %d len %d OOB %c ", Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> --- tools/testing/selftests/net/af_unix/test_unix_oob.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)