pass only "pinnable: this" to MemoryHandle

5 years ago · 289e9f8800
1 changed files with 3 additions and 1 deletions
--- a/src/ImageSharp/Memory/Allocators/Internals/ManagedBufferBase.cs
+++ b/src/ImageSharp/Memory/Allocators/Internals/ManagedBufferBase.cs
@ -24,7 +24,9 @@ namespace SixLabors.ImageSharp.Memory.Internals
            }

            void* ptr = (void*)this.pinHandle.AddrOfPinnedObject();
-            return new MemoryHandle(ptr, this.pinHandle, this);
+
+            // We should only pass pinnable:this, when GCHandle lifetime is managed by the MemoryManager<T> instance.
+            return new MemoryHandle(ptr, pinnable: this);
        }

        /// <inheritdoc />