[ros-diffs] [fireball] 48551: [NTOSKRNL] - Revert 48546. The code was correct, and there is more of same code in other places which firstly cancels the IRP and then moves to the next entry. The actual bug is somewhere else. See issue #5550 for more details.

Author: fireball
Date: Sun Aug 15 08:48:03 2010
New Revision: 48551

URL: http://svn.reactos.org/svn/reactos?rev=48551&view=rev
Log:
[NTOSKRNL]
- Revert 48546. The code was correct, and there is more of same code in other places which
firstly cancels the IRP and then moves to the next entry. The actual bug is somewhere
else.
See issue #5550 for more details.

Modified:
    trunk/reactos/ntoskrnl/io/iomgr/irp.c

Modified: trunk/reactos/ntoskrnl/io/iomgr/irp.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/irp.c?re…
==============================================================================
--- trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] Sun Aug 15 08:48:03 2010
@@ -1047,12 +1047,14 @@
     NextEntry = ListHead->Flink;
     while (ListHead != NextEntry)
     {
-        /* Get the IRP and move to the next entry */
+        /* Get the IRP */
         Irp = CONTAINING_RECORD(NextEntry, IRP, ThreadListEntry);
-        NextEntry = NextEntry->Flink;
 
         /* Cancel it */
         IoCancelIrp(Irp);
+
+        /* Move to the next entry */
+        NextEntry = NextEntry->Flink;
     }
 
      /* Wait 100 milliseconds */