[PATCH] msvcp120/tests: Check handle validity in test_thrd, not handle equality

Alex Henrie alexhenrie24 at gmail.com
Wed Dec 8 00:40:56 CST 2021


Fixes an intermittent testbot failure.

Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
 dlls/msvcp120/tests/msvcp120.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dlls/msvcp120/tests/msvcp120.c b/dlls/msvcp120/tests/msvcp120.c
index 495ee9dd6fb..6bc636c4b1f 100644
--- a/dlls/msvcp120/tests/msvcp120.c
+++ b/dlls/msvcp120/tests/msvcp120.c
@@ -2093,8 +2093,9 @@ static void test_thrd(void)
     tb = p__Thrd_current();
     ok(ta.id == tb.id, "got a %d b %d\n", ta.id, tb.id);
     ok(ta.id == GetCurrentThreadId(), "expected %d, got %d\n", GetCurrentThreadId(), ta.id);
-    /* these can be different if new threads are created at same time */
-    ok(ta.hnd == tb.hnd, "got a %p b %p\n", ta.hnd, tb.hnd);
+    /* the handles can be different if new threads are created at same time */
+    ok(ta.hnd != NULL, "handle a is NULL\n");
+    ok(tb.hnd != NULL, "handle b is NULL\n");
     ok(!CloseHandle(ta.hnd), "handle %p not closed\n", ta.hnd);
     ok(!CloseHandle(tb.hnd), "handle %p not closed\n", tb.hnd);
 
-- 
2.34.1




More information about the wine-devel mailing list