Return error instead of Null if i128 deserialized to Value is out of …

…range
serde-rs · Oct 18, 2024 · be68eab · be68eab
1 parent 95f55f3
commit be68eab
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/value/de.rs b/src/value/de.rs
@@ -44,9 +44,12 @@ impl<'de> Deserialize<'de> for Value {
                 Ok(Value::Number(value.into()))
             }
 
-            #[inline]
-            fn visit_i128<E>(self, value: i128) -> Result<Value, E> {
-                Ok(Number::from_i128(value).map_or(Value::Null, Value::Number))
+            fn visit_i128<E>(self, value: i128) -> Result<Value, E>
+            where
+                E: serde::de::Error,
+            {
+                let de = serde::de::value::I128Deserializer::new(value);
+                Number::deserialize(de).map(Value::Number)
             }
 
             #[inline]