[Win32k]
authorJames Tabor <james.tabor@reactos.org>
Sun, 10 Jan 2010 19:06:07 +0000 (19:06 +0000)
committerJames Tabor <james.tabor@reactos.org>
Sun, 10 Jan 2010 19:06:07 +0000 (19:06 +0000)
- When locking and unlocking regions, no need to use probe to check attribute space first before read or write access. The exception handling is a feature.

svn path=/trunk/; revision=45034

reactos/subsystems/win32/win32k/objects/region.c

index 36de953..7de49c2 100644 (file)
@@ -2085,7 +2085,6 @@ RGNOBJAPI_Lock(HRGN hRgn, PRGN_ATTR *ppRgn_Attr)
   PGDI_TABLE_ENTRY Entry;
   PROSRGNDATA pRgn;
   PRGN_ATTR pRgn_Attr;
-  BOOL Hit = FALSE;
 
   pRgn = REGION_LockRgn(hRgn);
 
@@ -2101,8 +2100,6 @@ RGNOBJAPI_Lock(HRGN hRgn, PRGN_ATTR *ppRgn_Attr)
      {
         _SEH2_TRY
         {
-           ProbeForWrite(pRgn_Attr, sizeof(RGN_ATTR), 1);
-
            if ( !(pRgn_Attr->AttrFlags & ATTR_CACHED) &&
                  pRgn_Attr->AttrFlags & (ATTR_RGN_VALID|ATTR_RGN_DIRTY) )
            {
@@ -2125,7 +2122,6 @@ RGNOBJAPI_Lock(HRGN hRgn, PRGN_ATTR *ppRgn_Attr)
         }
         _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
         {
-           Hit = TRUE;
         }
         _SEH2_END;
 
@@ -2148,7 +2144,6 @@ RGNOBJAPI_Unlock(PROSRGNDATA pRgn)
   INT Index;
   PGDI_TABLE_ENTRY Entry;
   PRGN_ATTR pRgn_Attr;
-  BOOL Hit = FALSE;
 
   if (pRgn)
   {
@@ -2162,8 +2157,6 @@ RGNOBJAPI_Unlock(PROSRGNDATA pRgn)
      {
         _SEH2_TRY
         {
-           ProbeForWrite(pRgn_Attr, sizeof(RGN_ATTR), 1);
-
            if ( pRgn_Attr->AttrFlags & ATTR_RGN_VALID )
            {
               pRgn_Attr->Flags = REGION_Complexity( pRgn );
@@ -2175,7 +2168,6 @@ RGNOBJAPI_Unlock(PROSRGNDATA pRgn)
         }
         _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
         {
-           Hit = TRUE;
         }
         _SEH2_END;
      }
@@ -2525,7 +2517,6 @@ REGION_SetRectRgn(
     }
     else
     {
-        DPRINT("SetRectRgn NULL Count: %d \n", rgn->rdh.nCount);
         EMPTY_REGION(rgn);
     }
 }