fix: skip number

cloudwego · Nov 8, 2024 · 25f4f87 · 25f4f87
1 parent f5ede7a
commit 25f4f87
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 16 deletions.
diff --git a/fuzz/.cargo/config b/fuzz/.cargo/config
diff --git a/scripts/fuzz.sh b/scripts/fuzz.sh
@@ -4,4 +4,4 @@ set -ex
 
 cargo install cargo-fuzz
 
-RUST_BACKTRACE=full cargo  +nightly fuzz run fuzz_value -- -max_total_time=20m
+RUST_BACKTRACE=full cargo  +nightly fuzz run fuzz_value 
diff --git a/src/parser.rs b/src/parser.rs
@@ -1262,7 +1262,8 @@ where
         }
 
         // SIMD path for long number
-        while let Some(chunk) = self.read.peek_n(32) {
+        const LANES: usize = i8x32::LANES;
+        while let Some(chunk) = self.read.peek_n(LANES) {
             let v = unsafe { i8x32::from_slice_unaligned_unchecked(chunk) };
             let zero = i8x32::splat(b'0' as i8);
             let nine = i8x32::splat(b'9' as i8);
@@ -1275,8 +1276,13 @@ where
                     // check the first digit after the dot
                     self.skip_single_digit()?;
 
-                    // check the remaining digits
+                    // check the overflow
                     cnt += 2;
+                    if cnt >= LANES {
+                        is_float = true;
+                        continue;
+                    }
+
                     nondigits = nondigits.wrapping_shr(cnt as u32);
                     if nondigits != 0 {
                         let offset = nondigits.trailing_zeros() as usize;

diff --git a/src/value/node.rs b/src/value/node.rs
@@ -2073,6 +2073,7 @@ mod test {
         use crate::Deserializer;
 
         let nums = [
+            "-46333333333333333333333333333333.6",
             "43.420273000",
             "1e123",
             "0.001","0e+12","0.1e+12",