Merge pull request #71 from solarwinds/NH-66479-int-conversion

NH-66479 address codeql scan finding
solarwinds · Mar 27, 2024 · d0efab5 · d0efab5
2 parents eccb96c + 2cf27a6
commit d0efab5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/internal/log/logging.go b/internal/log/logging.go
@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"io"
 	"log"
+	"math"
 	"os"
 	"path/filepath"
 	"runtime"
@@ -95,7 +96,7 @@ func ToLogLevel(level string) (LogLevel, bool) {
 
 	// Accept integers for backward-compatibility.
 	if i, err := strconv.Atoi(level); err == nil {
-		if i < len(LevelStr) {
+		if i >= 0 && i <= math.MaxInt8 && i < len(LevelStr) {
 			lvl = LogLevel(i)
 		} else {
 			return lvl, false