fix: possible out of bounds slice access

VirusTotal · Sep 6, 2023 · 7641544 · 7641544
1 parent 925695d
commit 7641544
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/yara-x/src/re/fast/fastvm.rs b/yara-x/src/re/fast/fastvm.rs
@@ -217,9 +217,11 @@ impl<'r> FastVM<'r> {
                     for position in &self.positions {
                         let jump_range =
                             *position..*position + step * jump as usize;
-                        if memchr::memchr(0x0A, &input[jump_range]).is_none() {
-                            next_positions
-                                .insert(position + step * jump as usize);
+                        if let Some(jump_range) = input.get(jump_range) {
+                            if memchr::memchr(0x0A, jump_range).is_none() {
+                                next_positions
+                                    .insert(position + step * jump as usize);
+                            }
                         }
                     }
                 }