[WIN32K]
authorJohannes Anderwald <johannes.anderwald@reactos.org>
Sat, 1 May 2010 10:42:45 +0000 (10:42 +0000)
committerJohannes Anderwald <johannes.anderwald@reactos.org>
Sat, 1 May 2010 10:42:45 +0000 (10:42 +0000)
- Add an assert to PostTimerMessages to catch null pointer windows
- Add another assert to catch special windows (Window == 1)
- Improve check if the window is '1'

svn path=/trunk/; revision=47070

reactos/subsystems/win32/win32k/ntuser/timer.c

index a1da2d6..1c34ab7 100644 (file)
@@ -279,7 +279,9 @@ PostTimerMessages(PWINDOW_OBJECT Window)
 
   if (!pTmr) return FALSE;
 
 
   if (!pTmr) return FALSE;
 
-  if (Window && (int)Window != 1)
+  ASSERT(Window);
+
+  if (Window && ((ULONG_PTR)Window != 1))
   {
      if (!Window->Wnd) return FALSE;
   }
   {
      if (!Window->Wnd) return FALSE;
   }
@@ -294,6 +296,7 @@ PostTimerMessages(PWINDOW_OBJECT Window)
           (pTmr->pti == pti) &&
           (pTmr->pWnd == Window))
         {
           (pTmr->pti == pti) &&
           (pTmr->pWnd == Window))
         {
+           ASSERT((ULONG_PTR)Window != 1);
            Msg.hwnd    = Window->hSelf;
            Msg.message = (pTmr->flags & TMRF_SYSTEM) ? WM_SYSTIMER : WM_TIMER;
            Msg.wParam  = (WPARAM) pTmr->nID;
            Msg.hwnd    = Window->hSelf;
            Msg.message = (pTmr->flags & TMRF_SYSTEM) ? WM_SYSTIMER : WM_TIMER;
            Msg.wParam  = (WPARAM) pTmr->nID;