[jimtabor] 45034: [Win32k] - 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.

Author: jimtabor
Date: Sun Jan 10 20:06:07 2010
New Revision: 45034

URL: http://svn.reactos.org/svn/reactos?rev=45034&view=rev
Log:
[Win32k]
- 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.

Modified:
    trunk/reactos/subsystems/win32/win32k/objects/region.c

Modified: trunk/reactos/subsystems/win32/win32k/objects/region.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/obj...
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/objects/region.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/objects/region.c [iso-8859-1] Sun Jan 10 20:06:07 2010
@@ -2085,7 +2085,6 @@
   PGDI_TABLE_ENTRY Entry;
   PROSRGNDATA pRgn;
   PRGN_ATTR pRgn_Attr;
-  BOOL Hit = FALSE;

pRgn = REGION_LockRgn(hRgn);

@@ -2101,8 +2100,6 @@
      {
         _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 @@
         }
         _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
         {
-           Hit = TRUE;
         }
         _SEH2_END;

@@ -2148,7 +2144,6 @@
   INT Index;
   PGDI_TABLE_ENTRY Entry;
   PRGN_ATTR pRgn_Attr;
-  BOOL Hit = FALSE;

if (pRgn)
   {
@@ -2162,8 +2157,6 @@
      {
         _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 @@
         }
         _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
         {
-           Hit = TRUE;
         }
         _SEH2_END;
      }
@@ -2525,7 +2517,6 @@
     }
     else
     {
-        DPRINT("SetRectRgn NULL Count: %d \n", rgn->rdh.nCount);
         EMPTY_REGION(rgn);
     }
 }