[ros-diffs] [cgutman] 37125: - Make sure ThunkTable is not NULL before attempting to free it - It gets set to NULL when the thunk is successfully added - I have no idea how this ever worked without this fix

cgutman at svn.reactos.org cgutman at svn.reactos.org
Sat Nov 1 02:00:09 CET 2008


Author: cgutman
Date: Fri Oct 31 20:00:08 2008
New Revision: 37125

URL: http://svn.reactos.org/svn/reactos?rev=37125&view=rev
Log:
 - Make sure ThunkTable is not NULL before attempting to free it
 - It gets set to NULL when the thunk is successfully added
 - I have no idea how this ever worked without this fix

Modified:
    trunk/reactos/ntoskrnl/mm/verifier.c

Modified: trunk/reactos/ntoskrnl/mm/verifier.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/verifier.c?rev=37125&r1=37124&r2=37125&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/verifier.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/verifier.c [iso-8859-1] Fri Oct 31 20:00:08 2008
@@ -146,7 +146,7 @@
     KeLeaveCriticalRegion();
 
     /* Free the table if we failed and return status */
-    ExFreePool(ThunkTable);
+    if (ThunkTable) ExFreePool(ThunkTable);
     return Status;
 }
 



More information about the Ros-diffs mailing list