Core/Logging: fixed log level range check in Log::SetLogLevel

Ovahlord · Ovahlord · commit 23a608a50492 · 2025-08-04T20:13:21.000+02:00
diff --git a/src/common/Logging/Log.cpp b/src/common/Logging/Log.cpp
@@ -262,10 +262,12 @@ std::string Log::GetTimestampStr()
 
 bool Log::SetLogLevel(std::string const& name, char const* newLevelc, bool isLogger /* = true */)
 {
-    LogLevel newLevel = LogLevel(atoi(newLevelc));
-    if (static_cast<int>(newLevel) < 0)
+    int32 newLeveli = atoi(newLevelc);
+    if (newLeveli < 0)
         return false;
 
+    LogLevel newLevel = static_cast<LogLevel>(newLeveli);
+
     if (isLogger)
     {
         auto it = loggers.begin();

Original file line number	Diff line number	Diff line change
`@@ -262,10 +262,12 @@ std::string Log::GetTimestampStr()`
`262`	`262`
`263`	`263`	`bool Log::SetLogLevel(std::string const& name, char const* newLevelc, bool isLogger /* = true */)`
`264`	`264`	`{`
`265`		`- LogLevel newLevel = LogLevel(atoi(newLevelc));`
`266`		`- if (static_cast<int>(newLevel) < 0)`
	`265`	`+ int32 newLeveli = atoi(newLevelc);`
	`266`	`+ if (newLeveli < 0)`
`267`	`267`	`return false;`
`268`	`268`
	`269`	`+ LogLevel newLevel = static_cast<LogLevel>(newLeveli);`
	`270`	`+`
`269`	`271`	`if (isLogger)`
`270`	`272`	`{`
`271`	`273`	`auto it = loggers.begin();`