int limits

Author: James Cor

enginetest/queries/script_queries.go

@@ -7799,6 +7799,26 @@ where
 			},
 		},
 	},
+	{
+		Name: "negative int limits",
+		Dialect: "mysql",
+		SetUpScript: []string{
+			"CREATE TABLE t(i8 tinyint, i16 smallint, i24 mediumint, i32 int, i64 bigint);",
+			"INSERT INTO t VALUES(-128, -32768, -8388608, -2147483648, -9223372036854775808);",
+		},
+		Assertions: []ScriptTestAssertion{
+			{
+				Query:    "SELECT -i8, -i16, -i24, -i32 from t;",
+				Expected: []sql.Row{
+					{128, 32768, 8388608, 2147483648},
+				},
+			},
+			{
+				Query:          "SELECT -i64 from t;",
+				ExpectedErrStr: "BIGINT out of range for -9223372036854775808",
+			},
+		},
+	},
 }
 
 var SpatialScriptTests = []ScriptTest{

sql/expression/arithmetic.go

@@ -16,7 +16,8 @@ package expression
 
 import (
 	"fmt"
-	"reflect"
+	"math"
+"reflect"
 	"regexp"
 	"strconv"
 	"strings"
@@ -704,12 +705,24 @@ func (e *UnaryMinus) Eval(ctx *sql.Context, row sql.Row) (interface{}, error) {
 	case int:
 		return -n, nil
 	case int8:
+		if n == math.MinInt8 {
+			return -int16(n), nil
+		}
 		return -n, nil
 	case int16:
+		if n == math.MinInt16 {
+			return -int32(n), nil
+		}
 		return -n, nil
 	case int32:
+		if n == math.MinInt32 {
+			return -int64(n), nil
+		}
 		return -n, nil
 	case int64:
+		if n == math.MinInt64 {
+			return nil, sql.ErrValueOutOfRange.New("BIGINT", fmt.Sprintf("%d", n))
+		}
 		return -n, nil
 	case uint:
 		return -int(n), nil