/W4: Fix incorrect instantiation of string transformer

The case transformation didn't work properly as the char value was not casted to an unsigned char, causing undefined behavior. Added new utility function 'IsEqualNoCase' which allows for comparing 2 strings case insensitively. Code has been tested with module strings containing irregular cases, and was still able to obtain the imported function address correctly.

r5dev/public/utility/module.cpp

@@ -343,10 +343,7 @@ CMemory CModule::GetImportedFunction(const string& svModuleName, const string& s
 		// Get virtual relative Address of the imported module name. Then add module base Address to get the actual location.
 		string svImportedModuleName = reinterpret_cast<char*>(reinterpret_cast<DWORD*>(m_pModuleBase + pIID->Name));
 
-		// Convert all characters to lower case because KERNEL32.DLL sometimes is kernel32.DLL, sometimes KERNEL32.dll.
-		std::transform(svImportedModuleName.begin(), svImportedModuleName.end(), svImportedModuleName.begin(), static_cast<int (*)(int)>(std::tolower));
-
-		if (svImportedModuleName.compare(svModuleName) == 0) // Is this our wanted imported module?.
+		if (IsEqualNoCase(svImportedModuleName, svModuleName)) // Is this our wanted imported module?.
 		{
 			// Original First Thunk to get function name.
 			IMAGE_THUNK_DATA* pOgFirstThunk = reinterpret_cast<IMAGE_THUNK_DATA*>(m_pModuleBase + pIID->OriginalFirstThunk);

r5dev/public/utility/utility.cpp

@@ -426,6 +426,17 @@ string ConvertToUnixPath(const string& svInput)
     return result;
 }
 
+///////////////////////////////////////////////////////////////////////////////
+// For comparing two strings (case insensitive).
+bool IsEqualNoCase(const string& svInput, const string& svSecond)
+{
+    return std::equal(svInput.begin(), svInput.end(), svSecond.begin(), svSecond.end(),
+        [](unsigned char ci, unsigned char cs)
+        {
+            return std::toupper(ci) == std::toupper(cs);
+        });
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // For checking if input is a valid Base64.
 bool IsValidBase64(const string& svInput, string* psvOutput)

r5dev/public/utility/utility.h

@@ -35,7 +35,9 @@ void CreateDirectories(string svInput, string* pszOutput = nullptr, bool bWindow
 string ConvertToWinPath(const string& svInput);
 string ConvertToUnixPath(const string& svInput);
 
+bool IsEqualNoCase(const string& svInput, const string& svSecond);
 bool IsValidBase64(const string& svInput, string* psvOutput = nullptr);
+
 string Base64Encode(const string& svInput);
 string Base64Decode(const string& svInput);