diff --git a/src/common.rs b/src/common.rs index ee94712..1987356 100644 --- a/src/common.rs +++ b/src/common.rs @@ -36,7 +36,12 @@ pub(crate) fn ceil_f64(v: f64) -> f64 { pub(crate) fn ceil_f64(v: f64) -> f64 { debug_assert!(v >= 0.0); - Decimal::from_f64(v).unwrap().ceil().to_f64().unwrap() + // `Decimal::from_f64` returns `None` for non-finite values, so fall back to + // the input and mirror `f64::ceil` instead of panicking. + match Decimal::from_f64(v) { + Some(d) => d.ceil().to_f64().unwrap_or(v), + None => v, + } } #[cfg(any(feature = "byte", feature = "bit"))] @@ -54,7 +59,12 @@ pub(crate) fn ceil_f32(v: f32) -> f32 { pub(crate) fn ceil_f32(v: f32) -> f32 { debug_assert!(v >= 0.0); - Decimal::from_f32(v).unwrap().ceil().to_f32().unwrap() + // `Decimal::from_f32` returns `None` for non-finite values, so fall back to + // the input and mirror `f32::ceil` instead of panicking. + match Decimal::from_f32(v) { + Some(d) => d.ceil().to_f32().unwrap_or(v), + None => v, + } } #[cfg(any(feature = "byte", feature = "bit"))] diff --git a/tests/byte.rs b/tests/byte.rs index 8687ce1..d0502c9 100644 --- a/tests/byte.rs +++ b/tests/byte.rs @@ -170,3 +170,12 @@ fn tests() { assert_eq!(byte, serde_json::from_str::(case.0).unwrap(), "{i}"); } } + +#[test] +fn from_non_finite_returns_none() { + // Non-finite inputs must return None instead of panicking, in both builds. + assert_eq!(Byte::from_f64(f64::INFINITY), None); + assert_eq!(Byte::from_f64(f64::NAN), None); + assert_eq!(Byte::from_f32(f32::INFINITY), None); + assert_eq!(Byte::from_f32(f32::NAN), None); +}