Amend the heap overflow fixes

GameConfig will need a bit of a rework but that doesn't belong in this branch, we've bloated it too much already.

src/gameconfig.cpp

@@ -244,8 +244,6 @@ int CGameConfig::HexStringToUint8Array(const char* hexString, uint8_t* byteArray
         }
     }
 
-    byteArray[byteCount] = '\0'; // Add a null-terminating character.
-
     return byteCount; // Return the number of bytes successfully converted.
 }
 
@@ -258,7 +256,7 @@ byte *CGameConfig::HexToByte(const char *src, size_t &length)
 	}
 
 	length = strlen(src) / 4;
-	uint8_t *dest = new uint8_t[length + 1];
+	uint8_t *dest = new uint8_t[length];
 	int byteCount = HexStringToUint8Array(src, dest, length);
 	if (byteCount <= 0)
 	{

src/utils/module.h

@@ -75,7 +75,7 @@ class CModule
 		for (size_t i = 0; i < m_size; i++)
 		{
 			size_t Matches = 0;
-			while (Matches < iSigLength && *(pMemory + i + Matches) == pData[Matches] || pData[Matches] == '\x2A')
+			while (*(pMemory + i + Matches) == pData[Matches] || pData[Matches] == '\x2A')
 			{
 				Matches++;
 				if (Matches == iSigLength)
@@ -87,6 +87,7 @@ class CModule
 					}
 
 					return_addr = (void *)(pMemory + i);
+					break;
 				}
 			}
 		}