[ion] 25003: - Properly set and check the image notification flag for the process in DbgkCreateThread. Patch by hto. See issue #1879 for more details.

Author: ion
Date: Thu Nov 30 22:52:49 2006
New Revision: 25003

URL: http://svn.reactos.org/svn/reactos?rev=25003&view=rev
Log:
- Properly set and check the image notification flag for the process in DbgkCreateThread. Patch by hto.
See issue #1879 for more details.

Modified:
    trunk/reactos/ntoskrnl/dbgk/dbgkutil.c
    trunk/reactos/ntoskrnl/tests/TestDebug.c

Modified: trunk/reactos/ntoskrnl/dbgk/dbgkutil.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/dbgk/dbgkutil.c?re...
==============================================================================
--- trunk/reactos/ntoskrnl/dbgk/dbgkutil.c (original)
+++ trunk/reactos/ntoskrnl/dbgk/dbgkutil.c Thu Nov 30 22:52:49 2006
@@ -104,10 +104,12 @@
     PTEB Teb;
     PAGED_CODE();

-    /* Check if this process has already been notified */
-    ProcessFlags = InterlockedAnd((PLONG)&Process->Flags,
-                                  PSF_CREATE_REPORTED_BIT |
-                                  PSF_IMAGE_NOTIFY_DONE_BIT);
+    /* Try ORing in the create reported and image notify flags */
+    ProcessFlags = InterlockedOr((PLONG)&Process->Flags,
+                                 PSF_CREATE_REPORTED_BIT |
+                                 PSF_IMAGE_NOTIFY_DONE_BIT);
+
+    /* Check if we were the first to set them or if another thread raced us */
     if (!(ProcessFlags & PSF_IMAGE_NOTIFY_DONE_BIT) && (PsImageNotifyEnabled))
     {
         /* It hasn't.. set up the image info for the process */

Modified: trunk/reactos/ntoskrnl/tests/TestDebug.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/tests/TestDebug.c?...
==============================================================================
--- trunk/reactos/ntoskrnl/tests/TestDebug.c (original)
+++ trunk/reactos/ntoskrnl/tests/TestDebug.c Thu Nov 30 22:52:49 2006
@@ -49,7 +49,7 @@
                            &ClientId);
     Status = DbgUiDebugActiveProcess(hProcess);
 #else
-    Error = DebugActiveProcess(2648);
+    Error = DebugActiveProcess(168);
 #endif

#if NATIVE