[ros-diffs] [cgutman] 38812: - Don't bugcheck if there is already an existing entry - This likely means that someone freed the memory without memtrack knowing (oskittcp does this sometimes)

[cgutman at svn.reactos.org]

Author: cgutman
Date: Sat Jan 17 07:33:59 2009
New Revision: 38812

URL: http://svn.reactos.org/svn/reactos?rev=38812&view=rev
Log:
 - Don't bugcheck if there is already an existing entry
 - This likely means that someone freed the memory without memtrack knowing (oskittcp does this sometimes)

Modified:
    branches/aicom-network-fixes/lib/drivers/ip/network/memtrack.c

Modified: branches/aicom-network-fixes/lib/drivers/ip/network/memtrack.c
URL: http://svn.reactos.org/svn/reactos/branches/aicom-network-fixes/lib/drivers/ip/network/memtrack.c?rev=38812&r1=38811&r2=38812&view=diff
==============================================================================
--- branches/aicom-network-fixes/lib/drivers/ip/network/memtrack.c [iso-8859-1] (original)
+++ branches/aicom-network-fixes/lib/drivers/ip/network/memtrack.c [iso-8859-1] Sat Jan 17 07:33:59 2009
@@ -82,14 +82,11 @@
 	if( ThingInList->Thing == Thing ) {
 	    RemoveEntryList(Entry);
 
-	    TcpipReleaseSpinLock( &AllocatedObjectsLock, OldIrql );
-	    ShowTrackedThing( "Alloc", ThingInList, FALSE );
+	    TrackDumpFL( FileName, LineNo );
 
-	    TrackDumpFL( FileName, LineNo );
-	    DbgPrint("TRACK: SPECIFIED ALREADY ALLOCATED ITEM %x\n", Thing);
-            ShowTrackedThing( "Double Alloc (Item in list)", ThingInList, TRUE );
-            ShowTrackedThing( "Double Alloc (Item not in list)", TrackedThing, TRUE );
-	    TcpipBugCheck( 0 );
+            //DbgPrint("TRACK: SPECIFIED ALREADY ALLOCATED ITEM %x\n", Thing);
+            ShowTrackedThing( "Double Alloc (Item in list)", ThingInList, FALSE );
+            ShowTrackedThing( "Double Alloc (Item not in list)", TrackedThing, FALSE );
 
             ExFreeToNPagedLookasideList( &AllocatedObjectsLookasideList,
 	                                 ThingInList );