Added null checks for Destructor call when exception occured during constructor (#156)

Johny404 · Jan Vícha · web-flow · commit ff22bd53be9c · 2025-02-04T17:41:13.000+01:00
Co-authored-by: Jan Vícha &lt;j.vicha@quadient.com&gt;
diff --git a/BTDB/StreamLayer/MemoryMappedMemReader.cs b/BTDB/StreamLayer/MemoryMappedMemReader.cs
@@ -82,9 +82,9 @@ public bool Eof(ref MemReader memReader)
 
     public void Dispose()
     {
-        _viewAccessor.SafeMemoryMappedViewHandle.ReleasePointer();
-        _memoryMappedFile.Dispose();
-        _fileHandle.Dispose();
+        _viewAccessor?.SafeMemoryMappedViewHandle.ReleasePointer();
+        _memoryMappedFile?.Dispose();
+        _fileHandle?.Dispose();
         GC.SuppressFinalize(this);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -82,9 +82,9 @@ public bool Eof(ref MemReader memReader)`
`82`	`82`
`83`	`83`	`public void Dispose()`
`84`	`84`	`{`
`85`		`- _viewAccessor.SafeMemoryMappedViewHandle.ReleasePointer();`
`86`		`- _memoryMappedFile.Dispose();`
`87`		`- _fileHandle.Dispose();`
	`85`	`+ _viewAccessor?.SafeMemoryMappedViewHandle.ReleasePointer();`
	`86`	`+ _memoryMappedFile?.Dispose();`
	`87`	`+ _fileHandle?.Dispose();`
`88`	`88`	`GC.SuppressFinalize(this);`
`89`	`89`	`}`
`90`	`90`