More exploit fixes by KIttenPopo (#126)

* Quick fix for a bug I caused

* Typo

* Update kitten-fixes branch to my repo (#122)

* Added string hash macro

* Added convenient vtfunc macro

* Made lil ConCommand creation macro

* Fixed multiple NET_SetConVar exploits

* Quick fixerino

* Fix convar struct (and other things)

* Revive clang-format (but good, i think)

* Update .clang-format

* Reformatted code to meet .clang-format requirements

* Minor formatting fixes

* Fixed Northstar "crashing" when console is closed

* Update .clang-format

* Quick fix for a bug I caused

* Typo

* NSMem Update

* ExplotFixes: Only block excessive convar counts if server

* Update ExploitFixes.cpp

* Update ExploitFixes.cpp

* Updated bytepatch format

* reformatted all code for clang-format

* Updated my clang-format to v13.0.0

* 3 fixes in 1

- ANTITAMPER fixed
- NSMem simplification update
- Fixed bad byte string in serverauthentication.cpp

* Improved ExploitFixes logging and NET_SetConVar patch

* clang-format unironically sabotaged my code

* Made ns_exploitfixes_log on by default

* Fixed IsMemoryReadable (oops)

Co-authored-by: KittenPopo <Pokeberry123@gmail.com>

1 files changed, 155 insertions, 155 deletions

diff --git a/NorthstarDedicatedTest/ExploitFixes_UTF8Parser.h b/NorthstarDedicatedTest/ExploitFixes_UTF8Parser.h
index b06d442b..09c26293 100644
--- a/NorthstarDedicatedTest/ExploitFixes_UTF8Parser.h
+++ b/NorthstarDedicatedTest/ExploitFixes_UTF8Parser.h
@@ -6,170 +6,170 @@
 
 namespace ExploitFixes_UTF8Parser
 {
-	bool __fastcall CheckValid(INT64* a1, DWORD* a2, char* strData)
-	{
-		static auto sub_F1320 = (INT64(__fastcall*)(DWORD a1, char* a2))NSMem::PatternScan("engine.dll", "83 F9 7F 77 08 88 0A");
+bool __fastcall CheckValid(INT64* a1, DWORD* a2, char* strData)
+{
+	static auto sub_F1320 = (INT64(__fastcall*)(DWORD a1, char* a2))NSMem::PatternScan("engine.dll", "83 F9 7F 77 08 88 0A");
 
-		DWORD v3;		// eax
-		char* v4;		// rbx
-		char v5;		// si
-		char* _strData; // rdi
-		char* v7;		// rbp
-		char v11;		// al
-		DWORD v12;		// er9
-		DWORD v13;		// ecx
-		DWORD v14;		// edx
-		DWORD v15;		// er8
-		int v16;		// eax
-		DWORD v17;		// er9
-		int v18;		// eax
-		DWORD v19;		// er9
-		DWORD v20;		// ecx
-		int v21;		// eax
-		int v22;		// er9
-		DWORD v23;		// edx
-		int v24;		// eax
-		int v25;		// er9
-		DWORD v26;		// er9
-		DWORD v27;		// er10
-		DWORD v28;		// ecx
-		DWORD v29;		// edx
-		DWORD v30;		// er8
-		int v31;		// eax
-		DWORD v32;		// er10
-		int v33;		// eax
-		DWORD v34;		// er10
-		DWORD v35;		// ecx
-		int v36;		// eax
-		int v37;		// er10
-		DWORD v38;		// edx
-		int v39;		// eax
-		int v40;		// er10
-		DWORD v41;		// er10
-		INT64 v43;		// r8
-		INT64 v44;		// rdx
-		INT64 v45;		// rcx
-		INT64 v46;		// rax
-		INT64 v47;		// rax
-		char v48;		// al
-		INT64 v49;		// r8
-		INT64 v50;		// rdx
-		INT64 v51;		// rcx
-		INT64 v52;		// rax
-		INT64 v53;		// rax
+	DWORD v3;		// eax
+	char* v4;		// rbx
+	char v5;		// si
+	char* _strData; // rdi
+	char* v7;		// rbp
+	char v11;		// al
+	DWORD v12;		// er9
+	DWORD v13;		// ecx
+	DWORD v14;		// edx
+	DWORD v15;		// er8
+	int v16;		// eax
+	DWORD v17;		// er9
+	int v18;		// eax
+	DWORD v19;		// er9
+	DWORD v20;		// ecx
+	int v21;		// eax
+	int v22;		// er9
+	DWORD v23;		// edx
+	int v24;		// eax
+	int v25;		// er9
+	DWORD v26;		// er9
+	DWORD v27;		// er10
+	DWORD v28;		// ecx
+	DWORD v29;		// edx
+	DWORD v30;		// er8
+	int v31;		// eax
+	DWORD v32;		// er10
+	int v33;		// eax
+	DWORD v34;		// er10
+	DWORD v35;		// ecx
+	int v36;		// eax
+	int v37;		// er10
+	DWORD v38;		// edx
+	int v39;		// eax
+	int v40;		// er10
+	DWORD v41;		// er10
+	INT64 v43;		// r8
+	INT64 v44;		// rdx
+	INT64 v45;		// rcx
+	INT64 v46;		// rax
+	INT64 v47;		// rax
+	char v48;		// al
+	INT64 v49;		// r8
+	INT64 v50;		// rdx
+	INT64 v51;		// rcx
+	INT64 v52;		// rax
+	INT64 v53;		// rax
 
-		v3 = a2[2];
-		v4 = (char*)(a1[1] + *a2);
-		v5 = 0;
-		_strData = strData;
-		v7 = &v4[*((UINT16*)a2 + 2)];
-		if (v3 >= 2)
+	v3 = a2[2];
+	v4 = (char*)(a1[1] + *a2);
+	v5 = 0;
+	_strData = strData;
+	v7 = &v4[*((UINT16*)a2 + 2)];
+	if (v3 >= 2)
+	{
+		++v4;
+		--v7;
+		if (v3 != 2)
 		{
-			++v4;
-			--v7;
-			if (v3 != 2)
+			while (1)
 			{
-				while (1)
-				{
 
-					if (!NSMem::IsMemoryReadable(v4, 1))
-						return false; // INVALID
+				if (!NSMem::IsMemoryReadable(v4, 1))
+					return false; // INVALID
 
-					v11 = *v4++; // crash potential
-					if (v11 != 92)
-						goto LABEL_6;
-					v11 = *v4++;
-					if (v11 == 110)
-						break;
-					switch (v11)
-					{
-					case 't':
-						v11 = 9;
-						goto LABEL_6;
-					case 'r':
-						v11 = 13;
-						goto LABEL_6;
-					case 'b':
-						v11 = 8;
-						goto LABEL_6;
-					case 'f':
-						v11 = 12;
-						goto LABEL_6;
-					}
-					if (v11 != 117)
-						goto LABEL_6;
-					v12 = *v4 | 0x20;
-					v13 = v4[1] | 0x20;
-					v14 = v4[2] | 0x20;
-					v15 = v4[3] | 0x20;
-					v16 = 87;
-					if (v12 <= 0x39)
-						v16 = 48;
-					v17 = v12 - v16;
-					v18 = 87;
-					v19 = v17 << 12;
-					if (v13 <= 0x39)
-						v18 = 48;
-					v20 = v13 - v18;
-					v21 = 87;
-					v22 = (v20 << 8) | v19;
-					if (v14 <= 0x39)
-						v21 = 48;
-					v23 = v14 - v21;
-					v24 = 87;
-					v25 = (16 * v23) | v22;
-					if (v15 <= 0x39)
-						v24 = 48;
-					v4 += 4;
-					v26 = (v15 - v24) | v25;
-					if (v26 - 55296 <= 0x7FF)
-					{
-						if (v26 >= 0xDC00)
-							return true;
-						if (*v4 != 92 || v4[1] != 117)
-							return true;
+				v11 = *v4++; // crash potential
+				if (v11 != 92)
+					goto LABEL_6;
+				v11 = *v4++;
+				if (v11 == 110)
+					break;
+				switch (v11)
+				{
+				case 't':
+					v11 = 9;
+					goto LABEL_6;
+				case 'r':
+					v11 = 13;
+					goto LABEL_6;
+				case 'b':
+					v11 = 8;
+					goto LABEL_6;
+				case 'f':
+					v11 = 12;
+					goto LABEL_6;
+				}
+				if (v11 != 117)
+					goto LABEL_6;
+				v12 = *v4 | 0x20;
+				v13 = v4[1] | 0x20;
+				v14 = v4[2] | 0x20;
+				v15 = v4[3] | 0x20;
+				v16 = 87;
+				if (v12 <= 0x39)
+					v16 = 48;
+				v17 = v12 - v16;
+				v18 = 87;
+				v19 = v17 << 12;
+				if (v13 <= 0x39)
+					v18 = 48;
+				v20 = v13 - v18;
+				v21 = 87;
+				v22 = (v20 << 8) | v19;
+				if (v14 <= 0x39)
+					v21 = 48;
+				v23 = v14 - v21;
+				v24 = 87;
+				v25 = (16 * v23) | v22;
+				if (v15 <= 0x39)
+					v24 = 48;
+				v4 += 4;
+				v26 = (v15 - v24) | v25;
+				if (v26 - 55296 <= 0x7FF)
+				{
+					if (v26 >= 0xDC00)
+						return true;
+					if (*v4 != 92 || v4[1] != 117)
+						return true;
 
-						v27 = v4[2] | 0x20;
-						v28 = v4[3] | 0x20;
-						v29 = v4[4] | 0x20;
-						v30 = v4[5] | 0x20;
-						v31 = 87;
-						if (v27 <= 0x39)
-							v31 = 48;
-						v32 = v27 - v31;
-						v33 = 87;
-						v34 = v32 << 12;
-						if (v28 <= 0x39)
-							v33 = 48;
-						v35 = v28 - v33;
-						v36 = 87;
-						v37 = (v35 << 8) | v34;
-						if (v29 <= 0x39)
-							v36 = 48;
-						v38 = v29 - v36;
-						v39 = 87;
-						v40 = (16 * v38) | v37;
-						if (v30 <= 0x39)
-							v39 = 48;
-						v4 += 6;
-						v41 = ((v30 - v39) | v40) - 56320;
-						if (v41 > 0x3FF)
-							return true;
-						v26 = v41 | ((v26 - 55296) << 10);
-					}
-					_strData += (DWORD)sub_F1320(v26, _strData);
-				LABEL_7:
-					if (v4 == v7)
-						goto LABEL_48;
+					v27 = v4[2] | 0x20;
+					v28 = v4[3] | 0x20;
+					v29 = v4[4] | 0x20;
+					v30 = v4[5] | 0x20;
+					v31 = 87;
+					if (v27 <= 0x39)
+						v31 = 48;
+					v32 = v27 - v31;
+					v33 = 87;
+					v34 = v32 << 12;
+					if (v28 <= 0x39)
+						v33 = 48;
+					v35 = v28 - v33;
+					v36 = 87;
+					v37 = (v35 << 8) | v34;
+					if (v29 <= 0x39)
+						v36 = 48;
+					v38 = v29 - v36;
+					v39 = 87;
+					v40 = (16 * v38) | v37;
+					if (v30 <= 0x39)
+						v39 = 48;
+					v4 += 6;
+					v41 = ((v30 - v39) | v40) - 56320;
+					if (v41 > 0x3FF)
+						return true;
+					v26 = v41 | ((v26 - 55296) << 10);
 				}
-				v11 = 10;
-			LABEL_6:
-				v5 |= v11;
-				*_strData++ = v11;
-				goto LABEL_7;
+				_strData += (DWORD)sub_F1320(v26, _strData);
+			LABEL_7:
+				if (v4 == v7)
+					goto LABEL_48;
 			}
+			v11 = 10;
+		LABEL_6:
+			v5 |= v11;
+			*_strData++ = v11;
+			goto LABEL_7;
 		}
-	LABEL_48:
-		return true;
 	}
+LABEL_48:
+	return true;
+}
 } // namespace ExploitFixes_UTF8Parser
\ No newline at end of file