[PATCH 0/1] MR1209: ntdll/tests: Fix uninitialized read in ok (Coverity)

[Fabian Maurer (＠DarkShadow44)]

Seems like a typo.
Also, there's no need to print the value of an always NULL handle.

Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de>

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1209

[PATCH 1/1] ntdll/tests: Fix uninitialized read in ok (Coverity)

[Fabian Maurer]

From: Fabian Maurer <dark.shadow4(a)web.de>

Seems like a typo.
Also, there's no need to print the value of an always NULL handle.

Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de>
---
 dlls/ntdll/tests/pipe.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/ntdll/tests/pipe.c b/dlls/ntdll/tests/pipe.c
index 6f26655b118..0e373dcf7ea 100644
--- a/dlls/ntdll/tests/pipe.c
+++ b/dlls/ntdll/tests/pipe.c
@@ -2920,13 +2920,13 @@ static void subtest_pipe_name(const struct pipe_name_test *pnt)
         return;
     }
 
-    ok(pipe != NULL, "expected non-NULL handle, got %p\n", client);
+    ok(pipe != NULL, "expected non-NULL handle\n");
 
     client = NULL;
     status = NtCreateFile(&client, SYNCHRONIZE, &attr, &iosb, NULL, 0,
                           FILE_SHARE_READ | FILE_SHARE_WRITE, FILE_OPEN, 0, NULL, 0);
     ok(status == STATUS_SUCCESS, "Expected success, got %#lx\n", status);
-    ok(client != NULL, "expected non-NULL handle, got %p\n", client);
+    ok(client != NULL, "expected non-NULL handle\n");
     NtClose(client);
 
     if (pnt->no_open_name)
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/1209