Skip to content

Commit

Permalink
Add fuzz target for Amount parsing
Browse files Browse the repository at this point in the history
  • Loading branch information
stevenroose committed May 17, 2019
1 parent a0bf7e8 commit adcc546
Show file tree
Hide file tree
Showing 2 changed files with 65 additions and 0 deletions.
4 changes: 4 additions & 0 deletions fuzz/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,10 @@ path = "fuzz_targets/deserialize_transaction.rs"
name = "deserialize_address"
path = "fuzz_targets/deserialize_address.rs"

[[bin]]
name = "deserialize_amount"
path = "fuzz_targets/deserialize_amount.rs"

[[bin]]
name = "deserialize_decimal"
path = "fuzz_targets/deserialize_decimal.rs"
Expand Down
61 changes: 61 additions & 0 deletions fuzz/fuzz_targets/deserialize_amount.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
extern crate bitcoin;
use std::str::FromStr;
fn do_test(data: &[u8]) {
let data_str = String::from_utf8_lossy(data);
let amt = match bitcoin::util::amount::Amount::from_str(&format!("{} BTC", data_str)) {
Ok(amt) => amt,
Err(_) => return,
};
let amt_roundtrip = match bitcoin::util::amount::Amount::from_str(&amt.to_string()) {
Ok(amt) => amt,
Err(_) => return,
};
assert_eq!(amt, amt_roundtrip);
}

#[cfg(feature = "afl")]
#[macro_use] extern crate afl;
#[cfg(feature = "afl")]
fn main() {
fuzz!(|data| {
do_test(&data);
});
}

#[cfg(feature = "honggfuzz")]
#[macro_use] extern crate honggfuzz;
#[cfg(feature = "honggfuzz")]
fn main() {
loop {
fuzz!(|data| {
do_test(data);
});
}
}

#[cfg(test)]
mod tests {
fn extend_vec_from_hex(hex: &str, out: &mut Vec<u8>) {
let mut b = 0;
for (idx, c) in hex.as_bytes().iter().enumerate() {
b <<= 4;
match *c {
b'A'...b'F' => b |= c - b'A' + 10,
b'a'...b'f' => b |= c - b'a' + 10,
b'0'...b'9' => b |= c - b'0',
_ => panic!("Bad hex"),
}
if (idx & 1) == 1 {
out.push(b);
b = 0;
}
}
}

#[test]
fn duplicate_crash() {
let mut a = Vec::new();
extend_vec_from_hex("00000000", &mut a);
super::do_test(&a);
}
}

0 comments on commit adcc546

Please sign in to comment.