[PATCH] msvcp120/tests: Compare thread IDs in test_thrd, not thread handles

Alex Henrie alexhenrie24 at gmail.com
Tue Dec 7 01:04:43 CST 2021


Fixes an intermittent testbot failure.

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

diff --git a/dlls/msvcp120/tests/msvcp120.c b/dlls/msvcp120/tests/msvcp120.c
index 495ee9dd6fb..3241e577b88 100644
--- a/dlls/msvcp120/tests/msvcp120.c
+++ b/dlls/msvcp120/tests/msvcp120.c
@@ -2093,8 +2093,8 @@ 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, but the IDs are always the same */
+    ok(GetThreadId(ta.hnd) == GetThreadId(tb.hnd), "got a %d b %d\n", GetThreadId(ta.hnd), GetThreadId(tb.hnd));
     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