[PATCH v2 4/4] server: Skip inaccessible threads in (get_next_thread).

[Paul Gofman]

From: Paul Gofman

---
 dlls/ntdll/tests/om.c |  3 +--
 server/thread.c       | 10 +++++++---
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c
index fd23038500a..bb6cf9059e1 100644
--- a/dlls/ntdll/tests/om.c
+++ b/dlls/ntdll/tests/om.c
@@ -3431,8 +3431,7 @@ static void test_zero_access(void)
     CloseHandle( h1 );

status = NtGetNextThread(GetCurrentProcess(), NULL, 0, 0, 0, &h1);
-    todo_wine ok( status == STATUS_NO_MORE_ENTRIES, "got %#lx.\n", status );
-    if (!status) CloseHandle( h1 );
+    ok( status == STATUS_NO_MORE_ENTRIES, "got %#lx.\n", status );

InitializeObjectAttributes( &attr, NULL, 0, 0, NULL );
     cid.UniqueProcess = ULongToHandle( GetCurrentProcessId() );
diff --git a/server/thread.c b/server/thread.c
index 0ba34511cd5..b3cf550349a 100644
--- a/server/thread.c
+++ b/server/thread.c
@@ -2023,9 +2023,13 @@ DECL_HANDLER(get_next_thread)
         thread = LIST_ENTRY( ptr, struct thread, entry );
         if (thread->process == process)
         {
-            reply->handle = alloc_handle( current->process, thread, req->access, req->attributes );
-            release_object( process );
-            return;
+            reply->handle = alloc_handle_user_open( current->process, thread, req->access, req->attributes );
+            if (get_error() != STATUS_ACCESS_DENIED)
+            {
+                release_object( process );
+                return;
+            }
+            clear_error();
         }
         ptr = req->flags ? list_prev( &thread_list, &thread->entry )
                          : list_next( &thread_list, &thread->entry );