Merge pull request #25 from trimble-oss/alert-autofix-197

Potential fix for code scanning alert no. 197: Incorrect conversion between integer types

Author: joel-rieke

sql/system_booltype.go

@@ -15,6 +15,7 @@
 package sql
 
 import (
+	"math"
 	"reflect"
 	"strconv"
 	"strings"
@@ -98,7 +99,10 @@ func (t systemBoolType) Convert(v interface{}) (interface{}, error) {
 		// Float values aren't truly accepted, but the engine will give them when it should give ints.
 		// Therefore, if the float doesn't have a fractional portion, we treat it as an int.
 		if value == float64(int64(value)) {
-			return t.Convert(int64(value))
+			if value >= float64(math.MinInt64) && value <= float64(math.MaxInt64) {
+				return t.Convert(int64(value))
+			}
+			return nil, ErrInvalidSystemVariableValue.New(t.varName, v)
 		}
 	case decimal.Decimal:
 		f, _ := value.Float64()