ntdll: Prevent NULL dereference in NtSuspendThread.

Overwatch calls NtSuspendThread directly, and expects to be able to pass in a NULL pointer for the count argument. Signed-off-by: Andrew Wesie <awesie@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

ntdll: Prevent NULL dereference in NtSuspendThread.
a39d67d8 · Andrew Wesie · Alexandre Julliard · e829de27 · a39d67d8
Commit a39d67d8 authored Feb 03, 2017 by Andrew Wesie Committed by Alexandre Julliard Feb 03, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

thread.c dlls/ntdll/thread.c +8 -2

No files found.
--- a/dlls/ntdll/thread.c
+++ b/dlls/ntdll/thread.c
@@ -632,7 +632,10 @@ NTSTATUS WINAPI NtSuspendThread( HANDLE handle, PULONG count )
    SERVER_START_REQ( suspend_thread )
    {
        req->handle = wine_server_obj_handle( handle );
-        if (!(ret = wine_server_call( req ))) *count = reply->count;
+        if (!(ret = wine_server_call( req )))
+        {
+            if (count) *count = reply->count;
+        }
    }
    SERVER_END_REQ;
    return ret;
@@ -650,7 +653,10 @@ NTSTATUS WINAPI NtResumeThread( HANDLE handle, PULONG count )
    SERVER_START_REQ( resume_thread )
    {
        req->handle = wine_server_obj_handle( handle );
-        if (!(ret = wine_server_call( req ))) *count = reply->count;
+        if (!(ret = wine_server_call( req )))
+        {
+            if (count) *count = reply->count;
+        }
    }
    SERVER_END_REQ;
    return ret;