Fix possible carsh for large negative integeR

Licenser · Licenser · commit 63fc4dfae3fb · 2019-07-28T19:18:50.000+02:00
diff --git a/data/crash/crash000030.json b/data/crash/crash000030.json
@@ -0,0 +1 @@
+{"Neh":-333333333333333333333333333333.3}
diff --git a/data/crash/crash000031.json b/data/crash/crash000031.json
@@ -0,0 +1 @@
+[  -9265394459000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.0p003]
diff --git a/data/crash/crash000032.json b/data/crash/crash000032.json
@@ -0,0 +1 @@
+[348253421170671280348253421170679.6171,-]
diff --git a/data/crash/crash000033.json b/data/crash/crash000033.json
@@ -0,0 +1 @@
+[3,28033333333333333333333333333333333348253421170679.6170-]
diff --git a/data/crash/crash000034.json b/data/crash/crash000034.json
@@ -0,0 +1,9 @@
+[3,2825342119.61701e-0,1,
+128253421170679.61701e-0,1,
+128253421170679.61701e-0,1,
+0.1e1,-1e-1,-128253421170679.61701e3,28253421170679.61701e-0,1,
+48253421170679.61701e-0,1,
+1266666666661,-128253421170679.61701e3,28253421170679.61701e-0,1,
+48253421170679.61701e-0,1,
+12666666666666666666666666666666666666666666666666666666666666666666666666666666666666666666666661170679.61701e3,28253421170679.61701e-0,1,
+48253421170679.61701e-0,16666666666666666666666666666666666666666666666666666666666666666666666679.6111-]
diff --git a/data/crash/crash000035.json b/data/crash/crash000035.json
@@ -0,0 +1 @@
+-9223372036854775808
diff --git a/data/crash/crash000036.json b/data/crash/crash000036.json
@@ -0,0 +1,5 @@
+[
+ 1066,
+1e1,
+0.1e1,
+1e-1,-1e-0, -88888888888888888888888888888888.1,"rod"]
diff --git a/src/numberparse.rs b/src/numberparse.rs
@@ -319,10 +319,11 @@ impl<'de> Deserializer<'de> {
         }
 
         if negative {
-            if i > 0x8000_0000_0000_0000 {
+            if i >= 9_223_372_036_854_775_808 {
+                //i64::min_value() * -1
                 return Err(self.error(ErrorType::Overflow));
             }
-        } else if i >= 0x8000_0000_0000_0000 {
+        } else if i > i64::max_value() as u64 {
             return Err(self.error(ErrorType::Overflow));
         }
 
diff --git a/tests/jsonchecker.rs b/tests/jsonchecker.rs
@@ -181,3 +181,14 @@ crash!(crash000026);
 crash!(crash000027);
 crash!(crash000028);
 crash!(crash000029);
+
+crash!(crash000030);
+crash!(crash000031);
+crash!(crash000032);
+crash!(crash000033);
+crash!(crash000034);
+crash!(crash000035);
+crash!(crash000036);
+// crash!(crash000037);
+// crash!(crash000038);
+// crash!(crash000039);

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+{"Neh":-333333333333333333333333333333.3}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+[ -9265394459000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.0p003]`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+[348253421170671280348253421170679.6171,-]`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+[3,28033333333333333333333333333333333348253421170679.6170-]`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +[
 + 1066,
 +1e1,
 +0.1e1,
 +1e-1,-1e-0, -88888888888888888888888888888888.1,"rod"]
Original file line number	Diff line number	Diff line change
`@@ -319,10 +319,11 @@ impl<'de> Deserializer<'de> {`
`319`	`319`	`}`
`320`	`320`
`321`	`321`	`if negative {`
`322`		`- if i > 0x8000_0000_0000_0000 {`
	`322`	`+ if i >= 9_223_372_036_854_775_808 {`
	`323`	`+ //i64::min_value() * -1`
`323`	`324`	`return Err(self.error(ErrorType::Overflow));`
`324`	`325`	`}`
`325`		`- } else if i >= 0x8000_0000_0000_0000 {`
	`326`	`+ } else if i > i64::max_value() as u64 {`
`326`	`327`	`return Err(self.error(ErrorType::Overflow));`
`327`	`328`	`}`
`328`	`329`