Replaced StringCchCopy usage with a for loop

src/loader/cxbxr-ldr.cpp

@@ -28,7 +28,6 @@
 
 #define WIN32_LEAN_AND_MEAN             // Exclude rarely-used stuff from Windows headers
 #include <Windows.h> // For LPTSTR, FormatMessage, GetSystemInfo, etc
-#include "strsafe.h" // For StringCchCopy
 
 #include "..\CxbxVersion.h"
 #include "..\Common\AddressRanges.h"
@@ -195,12 +194,14 @@ DWORD CALLBACK rawMain()
 		return ERROR_RESOURCE_NOT_FOUND;
 	}
 
-	// We cannot just pass the gui version of the loader via the Emulate function. This, because if the user mixes a version which does the check (and thus expects 3 arguments)
+	// We cannot just pass the git version of the loader via the Emulate function. This, because if the user mixes a version which does the check (and thus expects 3 arguments)
 	// with an old version which doesn't do the check (and thus only has 2 arguments), the behavior will be undefined since the new version will attempt to use a
 	// non-existent argument. We instead pass the version string in the contiguous memory, which must have been successfully reserved by now or else the loader would
 	// have already aborted execution. This memory is backed by the paging file, and thus its contents will always be initialized to zero. Thus, in the above scenerio
 	// the check will fail because a version string cannot be zero.
-	StringCchCopy(reinterpret_cast<char *>(PHYSICAL_MAP1_BASE + 0x1000), GitVersionLength + 1, CxbxGitVersion);
+	for (unsigned i = 0; i < GitVersionLength; ++i) {
+		*(reinterpret_cast<char *>(PHYSICAL_MAP1_BASE + 0x1000) + i) = CxbxGitVersion[i];
+	}
 
 	// Find the main emulation function in our DLL
 	typedef void (WINAPI *Emulate_t)(unsigned int, blocks_reserved_t);