[ros-diffs] [arty] 44223: io: don't need these references. data: pair references to section on map and unmap, pair references to file on section build, teardown and segment create and teardown.

Author: arty
Date: Wed Nov 18 08:13:08 2009
New Revision: 44223

URL: http://svn.reactos.org/svn/reactos?rev=44223&view=rev
Log:
io: don't need these references.
data: pair references to section on map and unmap, pair references to file on
section build, teardown and segment create and teardown.

Modified:
    branches/arty-newcc/ntoskrnl/mm/section/data.c
    branches/arty-newcc/ntoskrnl/mm/section/io.c

Modified: branches/arty-newcc/ntoskrnl/mm/section/data.c
URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/mm/section/d...
==============================================================================
--- branches/arty-newcc/ntoskrnl/mm/section/data.c [iso-8859-1] (original)
+++ branches/arty-newcc/ntoskrnl/mm/section/data.c [iso-8859-1] Wed Nov 18 08:13:08 2009
@@ -1715,6 +1715,7 @@
       ExAcquireFastMutex(&Segment->Lock);

DPRINT("Filling out Segment info (No previous data section)\n");
+	  ObReferenceObject(FileObject);
      Segment->FileObject = FileObject;
       Segment->Protection = SectionPageProtection;
       Segment->Flags = MM_DATAFILE_SEGMENT;
@@ -2201,6 +2202,7 @@
    	ASSERT(FALSE);
    }
    MiFreePageTablesSectionSegment(Segment);
+	ObDereferenceObject(Segment->FileObject);
    ExFreePool(Segment);

FileObject->SectionObjectPointer->DataSectionObject = NULL;
@@ -3078,6 +3080,9 @@

MmUnlockSectionSegment(Section->Segment);
    MmUnlockAddressSpace(AddressSpace);
+
+   if (Status == STATUS_SUCCESS)
+	   ObReferenceObject(Section);

return Status;
 }

Modified: branches/arty-newcc/ntoskrnl/mm/section/io.c
URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/mm/section/i...
==============================================================================
--- branches/arty-newcc/ntoskrnl/mm/section/io.c [iso-8859-1] (original)
+++ branches/arty-newcc/ntoskrnl/mm/section/io.c [iso-8859-1] Wed Nov 18 08:13:08 2009
@@ -263,8 +263,6 @@

Irp->Flags |= IRP_PAGING_IO | IRP_SYNCHRONOUS_PAGING_IO | IRP_NOCACHE | IRP_SYNCHRONOUS_API;

-    ObReferenceObject(FileObject);
-    
     Irp->UserEvent = &ReadWait;
     Irp->Tail.Overlay.OriginalFileObject = FileObject;
     Irp->Tail.Overlay.Thread = PsGetCurrentThread();
@@ -286,12 +284,9 @@
    	{
    		DPRINT1("Warning: Failed to wait for synchronous IRP\n");
    		ASSERT(FALSE);
-			ObDereferenceObject(FileObject);
    		return Status;
    	}
     }
-    
-    ObDereferenceObject(FileObject);

DPRINT("Paging IO Done: %08x\n", ReadStatus->Status);
    Status =