[ros-diffs] [hbelusca] 58121: [CSRSRV] Check for NULL parameter in the function CsrLocateThreadByClientId. Triggered when it was called by CsrApiHandleConnectionRequest during tests (the call to CsrLocateThrea...

Author: hbelusca
Date: Sun Jan  6 00:05:09 2013
New Revision: 58121

URL: http://svn.reactos.org/svn/reactos?rev=58121&view=rev
Log:
[CSRSRV]
Check for NULL parameter in the function CsrLocateThreadByClientId. Triggered when it was
called by CsrApiHandleConnectionRequest during tests (the call to
CsrLocateThreadByClientId happens in line 146 of api.c).

Modified:
    branches/ros-csrss/subsystems/win32/csrsrv/thredsup.c

Modified: branches/ros-csrss/subsystems/win32/csrsrv/thredsup.c
URL:
http://svn.reactos.org/svn/reactos/branches/ros-csrss/subsystems/win32/csrs…
==============================================================================
--- branches/ros-csrss/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] (original)
+++ branches/ros-csrss/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] Sun Jan  6 00:05:09
2013
@@ -188,6 +188,8 @@
     PCSR_THREAD FoundThread;
     // ASSERT(ProcessStructureListLocked());
 
+    if (Process) *Process = NULL;
+
     /* Hash the Thread */
     i = CsrHashThread(ClientId->UniqueThread);
 
@@ -210,7 +212,7 @@
              FoundThread->ClientId.UniqueThread  == ClientId->UniqueThread )
         {
             /* Match found, return the process */
-            *Process = FoundThread->Process;
+            if (Process) *Process = FoundThread->Process;
 
             /* Return thread too */
 //            DPRINT1("Found: %p %p\n", FoundThread, FoundThread->Process);