Merge pull request #1838 from UnderminersTeam/fix-erroneous-2022-1-de…

…tection Fix erroneous 2022.1 detection
UnderminersTeam · Jul 21, 2024 · 0334778 · 0334778
2 parents e5e0d5a + b6b29bf
commit 0334778
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 26 deletions.
diff --git a/UndertaleModLib/UndertaleChunks.cs b/UndertaleModLib/UndertaleChunks.cs
@@ -1010,7 +1010,7 @@ private void CheckForEffectData(UndertaleReader reader)
                     case LayerType.Assets:
                         reader.Position += 6 * 4;
                         int tileOffset = reader.ReadInt32();
-                        if (tileOffset != reader.AbsPosition + 8)
+                        if (tileOffset != reader.AbsPosition + 8 && tileOffset != reader.AbsPosition + 12)
                             reader.undertaleData.SetGMS2Version(2022, 1);
                         break;
                     case LayerType.Tiles:

diff --git a/UndertaleModLib/Util/BufferBinaryReader.cs b/UndertaleModLib/Util/BufferBinaryReader.cs
@@ -32,11 +32,7 @@ public int Read(byte[] buffer, int count)
                     n = count;
                 if (n <= 0)
                 {
-#if DEBUG
                     throw new IOException("Reading out of chunk bounds");
-#else
-                    return 0;
-#endif
                 }
 
                 if (n <= 8)
@@ -56,11 +52,7 @@ public int Read(Span<byte> buffer)
                 int n = Math.Min(_length - _position, buffer.Length);
                 if (n <= 0)
                 {
-#if DEBUG
                     throw new IOException("Reading out of chunk bounds");
-#else
-                    return 0;
-#endif
                 }
 
                 new Span<byte>(_buffer, _position, n).CopyTo(buffer);
@@ -74,11 +66,7 @@ public byte ReadByte()
                 int newPos = _position + 1;
                 if (newPos > _length)
                 {
-#if DEBUG
                     throw new IOException("Reading out of chunk bounds");
-#else
-                    return 0;
-#endif
                 }
 
                 _position = newPos;
@@ -186,10 +174,11 @@ public virtual bool ReadBoolean()
 
         public string ReadChars(int count)
         {
-#if DEBUG
             if (chunkBuffer.Position + count > _length)
+            {
                 throw new IOException("Reading out of chunk bounds");
-#endif
+            }
+
             if (count > 1024)
             {
                 byte[] buf = new byte[count];
@@ -209,10 +198,11 @@ public string ReadChars(int count)
 
         public byte[] ReadBytes(int count)
         {
-#if DEBUG
             if (chunkBuffer.Position + count > _length)
+            {
                 throw new IOException("Reading out of chunk bounds");
-#endif
+            }
+
             byte[] val = new byte[count];
             if (count > 0)
                 chunkBuffer.Read(val, count);
@@ -273,15 +263,14 @@ public ulong ReadUInt64()
 
         public string ReadGMString()
         {
-#if DEBUG
             if (chunkBuffer.Position + 5 > _length)
                 throw new IOException("Reading out of chunk bounds");
-#endif
+
             int length = BinaryPrimitives.ReadInt32LittleEndian(ReadToBuffer(4));
-#if DEBUG
+
             if (chunkBuffer.Position + length + 1 >= _length)
                 throw new IOException("Reading out of chunk bounds");
-#endif
+
             string res;
             if (length > 1024)
             {
@@ -296,13 +285,12 @@ public string ReadGMString()
                     chunkBuffer.Read(buf);
                 res = encoding.GetString(buf);
             }
-
-#if DEBUG
+
             if (ReadByte() != 0)
+            {
                 throw new IOException("String not null terminated!");
-#else
-            Position++;
-#endif
+            }
+
             return res;
         }
         public void SkipGMString()