[ros-diffs] [tkreuzer] 50893: [WIN32K] Fix possible lock leak in failure case of NtGdiSetBoundsRect. Fixes a failed assertion spotted by r3ddr4g0n.

tkreuzer at svn.reactos.org tkreuzer at svn.reactos.org
Thu Feb 24 14:18:55 UTC 2011


Author: tkreuzer
Date: Thu Feb 24 14:18:55 2011
New Revision: 50893

URL: http://svn.reactos.org/svn/reactos?rev=50893&view=rev
Log:
[WIN32K]
Fix possible lock leak in failure case of NtGdiSetBoundsRect. Fixes a failed assertion spotted by r3ddr4g0n.

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

Modified: trunk/reactos/subsystems/win32/win32k/objects/dcutil.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/objects/dcutil.c?rev=50893&r1=50892&r2=50893&view=diff
==============================================================================
--- trunk/reactos/subsystems/win32/win32k/objects/dcutil.c [iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/win32k/objects/dcutil.c [iso-8859-1] Thu Feb 24 14:18:55 2011
@@ -540,6 +540,7 @@
         }
         _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
         {
+            DC_UnlockDc(pdc);
             _SEH2_YIELD(return 0;)
         }
         _SEH2_END;
@@ -550,6 +551,6 @@
 
     if (flags & DCB_ENABLE) pdc->fs |= DC_ACCUM_APP;
     if (flags & DCB_DISABLE) pdc->fs &= ~DC_ACCUM_APP;
-    DC_UnlockDc( pdc );
+    DC_UnlockDc(pdc);
     return ret;
 }




More information about the Ros-diffs mailing list