[jgardou] 46939: [WIN32K] - move mouse related ppdev locking to Gre* functions Spotted by Physicus

Author: jgardou
Date: Mon Apr 19 16:17:26 2010
New Revision: 46939

URL: http://svn.reactos.org/svn/reactos?rev=46939&view=rev
Log:
[WIN32K]
  - move mouse related ppdev locking to Gre* functions
Spotted by Physicus

Modified:
    branches/reactos-yarotows/subsystems/win32/win32k/eng/mouse.c

Modified: branches/reactos-yarotows/subsystems/win32/win32k/eng/mouse.c
URL: http://svn.reactos.org/svn/reactos/branches/reactos-yarotows/subsystems/win3...
==============================================================================
--- branches/reactos-yarotows/subsystems/win32/win32k/eng/mouse.c [iso-8859-1] (original)
+++ branches/reactos-yarotows/subsystems/win32/win32k/eng/mouse.c [iso-8859-1] Mon Apr 19 16:17:26 2010
@@ -531,9 +531,7 @@
     if(ppdev->SafetyRemoveLevel)
         return ;

-    EngAcquireSemaphore(ppdev->hsemDevLock);
     ppdev->pfnMovePointer(pso, x, y, prcl);
-    EngReleaseSemaphore(ppdev->hsemDevLock);
 }

ULONG APIENTRY
@@ -555,7 +553,6 @@

pfnSetPointerShape = GDIDEVFUNCS(pso).SetPointerShape;

-    EngAcquireSemaphore(ppdev->hsemDevLock);
     if (pfnSetPointerShape)
     {
         ulResult = pfnSetPointerShape(pso,
@@ -593,8 +590,6 @@
         ppdev->pfnMovePointer = EngMovePointer;
     }

-    EngReleaseSemaphore(ppdev->hsemDevLock);
-
     return ulResult;
 }

@@ -623,11 +618,13 @@
     }

ASSERT(pdc->dctype == DCTYPE_DIRECT);
+    EngAcquireSemaphore(pdc->ppdev->hsemDevLock);
     /* We're not sure DC surface is the good one */
     psurf = pdc->ppdev->pSurface;
     if (!psurf)
     {
         DPRINT1("DC has no surface.\n");
+        EngReleaseSemaphore(pdc->ppdev->hsemDevLock);
         DC_UnlockDc(pdc);
         return 0;
     }
@@ -675,6 +672,8 @@
     if (psurfMask)
         SURFACE_ShareUnlockSurface(psurfMask);

+    EngReleaseSemaphore(pdc->ppdev->hsemDevLock);
+
     /* Unlock the DC */
     DC_UnlockDc(pdc);

@@ -701,12 +700,14 @@
     }
     ASSERT(pdc->dctype == DCTYPE_DIRECT);

+    EngAcquireSemaphore(pdc->ppdev->hsemDevLock);
     /* Store the cursor exclude position in the PDEV */
     prcl = &pdc->ppdev->Pointer.Exclude;

/* Call Eng/Drv function */
     IntEngMovePointer(&pdc->ppdev->pSurface->SurfObj, x, y, prcl);

+    EngReleaseSemaphore(pdc->ppdev->hsemDevLock);
     /* Unlock the DC */
     DC_UnlockDc(pdc);
 }