From d09b29c3e8776efa1376e240312d8802b1b8f5dd Mon Sep 17 00:00:00 2001
From: Hartmut Birr <osexpert@googlemail.com>
Date: Mon, 25 Aug 2003 09:14:09 +0000
Subject: [PATCH] - Check always the irq trapframe's cs before delivering an
 user mode apc. - Copy the irq trap frame only for apc's.

svn path=/trunk/; revision=5843
---
 reactos/ntoskrnl/ke/i386/irq.c | 45 ++++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/reactos/ntoskrnl/ke/i386/irq.c b/reactos/ntoskrnl/ke/i386/irq.c
index 634bb6bc76f..425fc592d12 100644
--- a/reactos/ntoskrnl/ke/i386/irq.c
+++ b/reactos/ntoskrnl/ke/i386/irq.c
@@ -16,7 +16,7 @@
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
-/* $Id: irq.c,v 1.34 2003/07/20 12:14:46 dwelch Exp $
+/* $Id: irq.c,v 1.35 2003/08/25 09:14:09 hbirr Exp $
  *
  * PROJECT:         ReactOS kernel
  * FILE:            ntoskrnl/ke/i386/irq.c
@@ -485,13 +485,6 @@ KiInterruptDispatch (ULONG irq, PKIRQ_TRAPFRAME Trapframe)
        return;
      }
 
-   CurrentThread = KeGetCurrentThread();
-   if (NULL != CurrentThread && NULL == CurrentThread->TrapFrame)
-      {
-	OldTrapFrame = CurrentThread->TrapFrame;
-	KeIRQTrapFrameToTrapFrame(Trapframe, &KernelTrapFrame);
-	CurrentThread->TrapFrame = &KernelTrapFrame;
-      }
 
    /*
     * Enable interrupts
@@ -526,19 +519,33 @@ KiInterruptDispatch (ULONG irq, PKIRQ_TRAPFRAME Trapframe)
    __asm__("cli\n\t");
    HalEndSystemInterrupt (old_level, 0);
 
-   if (old_level==PASSIVE_LEVEL && CurrentThread!=NULL &&
-       CurrentThread->Alerted[1] != 0 &&
-       CurrentThread->TrapFrame->Cs != KERNEL_CS)
+   if (old_level==PASSIVE_LEVEL && Trapframe->Cs != KERNEL_CS)
      {
-       KiDeliverNormalApc();
+       CurrentThread = KeGetCurrentThread();
+       if (CurrentThread!=NULL && CurrentThread->Alerted[1])
+         {
+           DPRINT("PID: %d, TID: %d CS %04x/%04x\n", 
+	          ((PETHREAD)CurrentThread)->ThreadsProcess->UniqueProcessId,
+		  ((PETHREAD)CurrentThread)->Cid.UniqueThread,
+		  Trapframe->Cs, 
+		  CurrentThread->TrapFrame ? CurrentThread->TrapFrame->Cs : 0);
+	   if (CurrentThread->TrapFrame == NULL)
+	     {
+	       OldTrapFrame = CurrentThread->TrapFrame;
+	       KeIRQTrapFrameToTrapFrame(Trapframe, &KernelTrapFrame);
+	       CurrentThread->TrapFrame = &KernelTrapFrame;
+	     }
+	   
+           KiDeliverNormalApc();
+           
+	   assert(KeGetCurrentThread() == CurrentThread);
+           if (CurrentThread->TrapFrame == &KernelTrapFrame)
+	     {
+               KeTrapFrameToIRQTrapFrame(&KernelTrapFrame, Trapframe);
+	       CurrentThread->TrapFrame = OldTrapFrame;
+	     }
+	 }
      }
-
-   assert(KeGetCurrentThread() == CurrentThread);
-   if (NULL != CurrentThread && CurrentThread->TrapFrame == &KernelTrapFrame)
-      {
-	KeTrapFrameToIRQTrapFrame(&KernelTrapFrame, Trapframe);
-	CurrentThread->TrapFrame = OldTrapFrame;
-      }
 }
 
 #endif /* MP */