Merge pull request #40 from killercup/feature/fuzz

Add simple parser fuzzer

fuzz/.gitignore

@@ -0,0 +1,4 @@
+
+target
+corpus
+artifacts

fuzz/Cargo.toml

@@ -0,0 +1,22 @@
+
+[package]
+name = "Molten-fuzz"
+version = "0.0.1"
+authors = ["Automatically generated"]
+publish = false
+
+[package.metadata]
+cargo-fuzz = true
+
+[dependencies.Molten]
+path = ".."
+[dependencies.libfuzzer-sys]
+git = "https://github.com/rust-fuzz/libfuzzer-sys.git"
+
+# Prevent this from interfering with workspaces
+[workspace]
+members = ["."]
+
+[[bin]]
+name = "parse"
+path = "fuzz_targets/parse.rs"

fuzz/fuzz_targets/parse.rs

@@ -0,0 +1,9 @@
+#![no_main]
+#[macro_use] extern crate libfuzzer_sys;
+extern crate Molten;
+
+fuzz_target!(|data: &[u8]| {
+    if let Ok(text) = ::std::str::from_utf8(data) {
+        let _ = Molten::parser::Parser::new(text).parse();
+    }
+});

src/parser.rs

@@ -319,7 +319,7 @@ impl<'a> Parser<'a> {
             '"' => self.parse_basic_string(),
             '\'' => self.parse_literal_string(),
             // Boolean: true
-            't' if &self.src[self.idx..self.idx + 4] == "true" => {
+            't' if self.src[self.idx..].starts_with("true") => {
                 self.inc();
                 self.inc();
                 self.inc();
@@ -331,7 +331,7 @@ impl<'a> Parser<'a> {
                 })
             }
             // Boolean: False
-            'f' if &self.src[self.idx..self.idx + 5] == "false" => {
+            'f' if self.src[self.idx..].starts_with("false") => {
                 self.inc();
                 self.inc();
                 self.inc();
@@ -744,4 +744,16 @@ mod tests {
         invalid_ints.iter().for_each(|s| {assert_eq!(None, Parser::parse_number(s, Trivia::default()))});
         invalid_floats.iter().for_each(|s| {assert_eq!(None, Parser::parse_number(s, Trivia::default()))});
     }
+
+    #[test]
+    fn issue41() {
+        let text = ::std::str::from_utf8(b"\'\'fb\'\xee\x9d\xbd").unwrap();
+        let _ = Parser::new(text).parse();
+    }
+
+    #[test]
+    fn issue42() {
+        let text = ::std::str::from_utf8(b"\'\nv\'f%\nb").unwrap();
+        let _ = Parser::new(text).parse();
+    }
 }