fix: Don't allocate extraneous memory in FastBufferReader (#2265)

* fix: Don't allocate extraneous memory in FastBufferReader

* Add PR number to CHANGELOG entry

Author: simon-lemay-unity

com.unity.netcode.gameobjects/CHANGELOG.md

@@ -7,6 +7,12 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 
 Additional documentation and release notes are available at [Multiplayer Documentation](https://docs-multiplayer.unity3d.com).
 
+## [Unreleased]
+
+### Fixed
+
+- Creating a `FastBufferReader` with `Allocator.None` will not result in extra memory being allocated for the buffer (since it's owned externally in that scenario). (#2265)
+
 ## [1.1.0] - 2022-10-21
 
 ### Added

com.unity.netcode.gameobjects/Runtime/Serialization/FastBufferReader.cs

@@ -65,7 +65,7 @@ internal unsafe void CommitBitwiseReads(int amount)
             ReaderHandle* readerHandle = null;
             if (copyAllocator == Allocator.None)
             {
-                readerHandle = (ReaderHandle*)UnsafeUtility.Malloc(sizeof(ReaderHandle) + length, UnsafeUtility.AlignOf<byte>(), internalAllocator);
+                readerHandle = (ReaderHandle*)UnsafeUtility.Malloc(sizeof(ReaderHandle), UnsafeUtility.AlignOf<byte>(), internalAllocator);
                 readerHandle->BufferPointer = buffer;
                 readerHandle->Position = offset;
             }