Merge pull request #24 from trimble-oss/alert-autofix-221

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

Author: joel-rieke

sql/system_settype.go

@@ -15,6 +15,7 @@
 package sql
 
 import (
+	"math"
 	"reflect"
 
 	"github.com/dolthub/vitess/go/sqltypes"
@@ -90,6 +91,9 @@ func (t systemSetType) 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)) {
+			if value < float64(math.MinInt64) || value > float64(math.MaxInt64) {
+				return nil, ErrInvalidSystemVariableValue.New(t.varName, v)
+			}
 			return t.SetType.Convert(int64(value))
 		}
 	case decimal.Decimal: