From 696036219e6d545ac844a98729df32a5841f4c8d Mon Sep 17 00:00:00 2001
From: Alexander 'z33ky' Hirsch <1zeeky@gmail.com>
Date: Wed, 13 Nov 2024 00:25:51 +0100
Subject: [PATCH] Prevent signed overflow (UB) in
 SquirrelVM::GenerateUniqueKey()

---
 sp/src/vscript/vscript_squirrel.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sp/src/vscript/vscript_squirrel.cpp b/sp/src/vscript/vscript_squirrel.cpp
index 5efb2314..9f46d68c 100644
--- a/sp/src/vscript/vscript_squirrel.cpp
+++ b/sp/src/vscript/vscript_squirrel.cpp
@@ -2776,7 +2776,7 @@ void* SquirrelVM::GetInstanceValue(HSCRIPT hInstance, ScriptClassDesc_t* pExpect
 
 bool SquirrelVM::GenerateUniqueKey(const char* pszRoot, char* pBuf, int nBufSize)
 {
-	static int keyIdx = 0;
+	static unsigned keyIdx = 0;
 	// This gets used for script scope, still confused why it needs to be inside IScriptVM
 	// is it just to be a compatible name for CreateScope?
 	V_snprintf(pBuf, nBufSize, "%08X_%s", ++keyIdx, pszRoot);