Merge pull request #607 from TheBlueMatt/2020-04-more-cleanups
authorMatt Corallo <649246+TheBlueMatt@users.noreply.github.com>
Fri, 24 Apr 2020 22:17:53 +0000 (22:17 +0000)
committerGitHub <noreply@github.com>
Fri, 24 Apr 2020 22:17:53 +0000 (22:17 +0000)
CI and rustc warning cleanup

34 files changed:
.github/workflows/build.yml
fuzz/ci-fuzz.sh
fuzz/src/bin/chanmon_consistency_target.rs
fuzz/src/bin/chanmon_deser_target.rs
fuzz/src/bin/full_stack_target.rs
fuzz/src/bin/msg_accept_channel_target.rs
fuzz/src/bin/msg_announcement_signatures_target.rs
fuzz/src/bin/msg_channel_announcement_target.rs
fuzz/src/bin/msg_channel_reestablish_target.rs
fuzz/src/bin/msg_channel_update_target.rs
fuzz/src/bin/msg_closing_signed_target.rs
fuzz/src/bin/msg_commitment_signed_target.rs
fuzz/src/bin/msg_decoded_onion_error_packet_target.rs
fuzz/src/bin/msg_error_message_target.rs
fuzz/src/bin/msg_funding_created_target.rs
fuzz/src/bin/msg_funding_locked_target.rs
fuzz/src/bin/msg_funding_signed_target.rs
fuzz/src/bin/msg_init_target.rs
fuzz/src/bin/msg_node_announcement_target.rs
fuzz/src/bin/msg_onion_hop_data_target.rs
fuzz/src/bin/msg_open_channel_target.rs
fuzz/src/bin/msg_ping_target.rs
fuzz/src/bin/msg_pong_target.rs
fuzz/src/bin/msg_revoke_and_ack_target.rs
fuzz/src/bin/msg_shutdown_target.rs
fuzz/src/bin/msg_update_add_htlc_target.rs
fuzz/src/bin/msg_update_fail_htlc_target.rs
fuzz/src/bin/msg_update_fail_malformed_htlc_target.rs
fuzz/src/bin/msg_update_fee_target.rs
fuzz/src/bin/msg_update_fulfill_htlc_target.rs
fuzz/src/bin/peer_crypt_target.rs
fuzz/src/bin/router_target.rs
fuzz/src/bin/target_template.txt
lightning/src/ln/channel.rs

index 98f4f19a4d6a898f78859c2aa4d1466beee1cf88..cd9b9f63553609cab54fe8d25b474d6db40b97d8 100644 (file)
@@ -1,7 +1,6 @@
 name: Continuous Integration Checks
 
-on:
-  push:
+on: [push, pull_request]
 
 jobs:
   build:
@@ -91,7 +90,7 @@ jobs:
         run: |
           sudo apt-get update
           sudo apt-get -y install build-essential binutils-dev libunwind-dev
-      - name: Fuzz test on Rust ${{ matrix.TOOLCHAIN }}
+      - name: Sanity check fuzz targets on Rust ${{ env.TOOLCHAIN }}
         run: cd fuzz && cargo test --verbose --color always
-      - name: Generate fuzz report
+      - name: Run fuzzers
         run: cd fuzz && ./ci-fuzz.sh
index 57e326472886e654af93503c88b5611f7a9e2e2a..9894178458da6afdac246500474ef70014588f23 100755 (executable)
@@ -12,20 +12,22 @@ rm *_target.rs
 [ "$(git diff)" != "" ] && exit 1
 popd
 
-cargo install --force honggfuzz
+cargo install --color always --force honggfuzz
 sed -i 's/lto = true//' Cargo.toml
-HFUZZ_BUILD_ARGS="--features honggfuzz_fuzz" cargo hfuzz build
+HFUZZ_BUILD_ARGS="--features honggfuzz_fuzz" cargo --color always hfuzz build
 for TARGET in src/bin/*.rs; do
        FILENAME=$(basename $TARGET)
        FILE="${FILENAME%.*}"
        HFUZZ_RUN_ARGS="--exit_upon_crash -v -n2"
        if [ "$FILE" = "chanmon_consistency_target" ]; then
                HFUZZ_RUN_ARGS="$HFUZZ_RUN_ARGS -F 64 -N100000"
+       elif [ "$FILE" = "full_stack_target" ]; then
+               HFUZZ_RUN_ARGS="$HFUZZ_RUN_ARGS -t0 -N1000000"
        else
                HFUZZ_RUN_ARGS="$HFUZZ_RUN_ARGS -N1000000"
        fi
        export HFUZZ_RUN_ARGS
-       HFUZZ_BUILD_ARGS="--features honggfuzz_fuzz" cargo hfuzz run $FILE
+       HFUZZ_BUILD_ARGS="--features honggfuzz_fuzz" cargo --color always hfuzz run $FILE
        if [ -f hfuzz_workspace/$FILE/HONGGFUZZ.REPORT.TXT ]; then
                cat hfuzz_workspace/$FILE/HONGGFUZZ.REPORT.TXT
                for CASE in hfuzz_workspace/$FILE/SIG*; do
index 845503da1a4827171b80f7c61181d84ed4186522..ff082497a67bbe09ea274e9033ede213ce127a30 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::chanmon_consistency::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        chanmon_consistency_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 1677721b2ce641d369171e36d79714d71bd57e2b..45df25199e1b29f395f5a7448a4a03a6acfe9865 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::chanmon_deser::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        chanmon_deser_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 94da2888fa8a6da5f8e8e59893daafbff7381b93..7ad5d148ecd4c562b0da6744ff76bca38734a559 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::full_stack::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        full_stack_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index f7bfc10d0d68898ffb2f9bba556561cdbcc428b8..dcd799af3c03ad1ba5db42c8bbca841605d2c2f6 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_accept_channel::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_accept_channel_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 63c6407eb26587b414306d57bf5d18f2431b3f3a..60124e928c2f2eec16c5e17e1b0d2a9bc0f601e8 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_announcement_signatures::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_announcement_signatures_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 8aae2770234a515bf0daf12ab84a68f8e2a3fd0c..c20a62a3f9369827ae8ce1e3e4e9696e7458f63c 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_channel_announcement::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_channel_announcement_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index fb5e4774b8c0f80db1cd0d77251286c22dc37409..99301da6b739db7ff0b2988c589d85314fff5fe6 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_channel_reestablish::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_channel_reestablish_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 78c6404f37c345ca4a53d91a7ca9b7f130065256..2ccc1e3fa3d3f0d9def941d341081eb2ced940f7 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_channel_update::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_channel_update_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 97b49ca8d49aa2d8ea2ecb1d7959334e7d9ea8b2..5e676be39141e81dac3b79f6c38e1ba8045aa9da 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_closing_signed::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_closing_signed_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index a343e45f7d1bf0f5b9286dafe1ddae3047634f55..c37f0030ee48b98534cf54ae65683d0e6f91b1fa 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_commitment_signed::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_commitment_signed_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 26e26fac9ba2388465c4217e6627e7406b5c5667..ae025cc68f9bc1dbb09ea53e5461f3e94eae51d9 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_decoded_onion_error_packet::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_decoded_onion_error_packet_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 34facca3fa95816f8dfdae4fe110555b981d9a6b..a72fe53339fb9863a1abec8dfa216d482a3a2a49 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_error_message::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_error_message_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 602f99067a011cc8f15b1e5e816514fad481a50c..4e885b64684b7994b36d88e291a3481f635a9e56 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_funding_created::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_funding_created_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index ee44753de9847cd4d881f355669079058a355ed1..0dc343c67db5af892c5c6c877e2d1efe56e93b16 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_funding_locked::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_funding_locked_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index d61da65fa655e275b2abd0a2322d4b1354ada505..0fb95103515a165e2a96b61446cb818fb54233f4 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_funding_signed::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_funding_signed_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 62b4e4159d82848528e44559eb925fe17b1545e6..707305231320e991b6270acad72350fd02dfc0d4 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_init::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_init_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 4605f9495a847c07afd79c5aa618b9f9bed4aaf9..1a5484169d4954e282dce91735e0b751dde1a1cf 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_node_announcement::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_node_announcement_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 1dcb5c7d550b56bbf10e63d974e7fbbbf92fb9b9..75a5c19c2715cb0745be4c81c91d92caabb73373 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_onion_hop_data::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_onion_hop_data_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 96fed7f33030accaee3ee12debb37920176aef59..f5843b7c36db0978f465b287cc13da34d869640a 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_open_channel::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_open_channel_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index cf2cacf2dcbe63ea4a018f5883b8b5f7dbb909b5..b0dd707ea670c96b168e11387eeb2d28b229c0cf 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_ping::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_ping_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index c538d1383f22ce6a4369e25db330f3c1b6cbe947..6f9ae4e06bd94ea44083f2a7d473d4c8b3d3cef0 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_pong::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_pong_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 08fe92a2ad3c759aed9938d442bb2b4956617323..dcf3e99f8c89c52626c36aded69bc74191e31af5 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_revoke_and_ack::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_revoke_and_ack_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 647e7cc7be39530c81a4b546138ba837f6e9bcc1..cb6f4e8d7c0569bb9dda7fe34c1f40fc0f4bc85f 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_shutdown::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_shutdown_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index abb749a68d36f32ba4846933dee688d7d09d7e1f..b4b9bc9f34dda28709458eefa861714fb0d640c6 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_update_add_htlc::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_update_add_htlc_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 278df1fe822b74fff883a798383b1de333a9e617..557c22912e08db540fdc1af2b01bf729f2f5ffb3 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_update_fail_htlc::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_update_fail_htlc_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 8c6b1e998e976efa675b9c0cfbecf5ec1714f67b..c62aa7633f65d6ee9d1a22dfea39a1679a5067d7 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_update_fail_malformed_htlc::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_update_fail_malformed_htlc_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 2b01b2b6b7cfab3b670de5cadc8c86609e21611e..887843a658424b9cb8c798e8c556d529e3e596c7 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_update_fee::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_update_fee_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index dbe56d7b57b2858054025bd898b29c84062075ae..1f9d9c910a3bd11f500f4dd976c8defccfec70ff 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::msg_targets::msg_update_fulfill_htlc::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        msg_update_fulfill_htlc_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index f68eaf1b28b004a1371517b64c8b49954aabb884..b30c33ab2afa4fa9a261dba92f2a405701b66e97 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::peer_crypt::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        peer_crypt_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index 8ab074b58353db9cf2935edd2c1f644f0ca0b464..588c811c65da86c951e641b4680d44c970ca8b26 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::router::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        router_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index ae2130a9cc2c314fb9fea28bf44578a3be33e1a7..2dcf22712cb485042536d4de1f1acce5e69e2206 100644 (file)
@@ -6,9 +6,6 @@
 extern crate lightning_fuzz;
 use lightning_fuzz::TARGET_MOD::*;
 
-use std::fs;
-use std::io::Read;
-
 #[cfg(feature = "afl")]
 #[macro_use] extern crate afl;
 #[cfg(feature = "afl")]
@@ -38,6 +35,8 @@ fuzz_target!(|data: &[u8]| {
 
 #[cfg(feature = "stdin_fuzz")]
 fn main() {
+       use std::io::Read;
+
        let mut data = Vec::with_capacity(8192);
        std::io::stdin().read_to_end(&mut data).unwrap();
        TARGET_NAME_run(data.as_ptr(), data.len());
@@ -45,6 +44,8 @@ fn main() {
 
 #[test]
 fn run_test_cases() {
+       use std::fs;
+       use std::io::Read;
        use lightning_fuzz::utils::test_logger::StringBuffer;
 
        use std::sync::{atomic, Arc};
index f44109432a581c2f28a7b3797cc187d24546ed31..740f67584deceaad3c2463687a8ccef24520f841 100644 (file)
@@ -207,8 +207,8 @@ enum ChannelState {
        /// to drop us, but we store this anyway.
        ShutdownComplete = 4096,
 }
-const BOTH_SIDES_SHUTDOWN_MASK: u32 = (ChannelState::LocalShutdownSent as u32 | ChannelState::RemoteShutdownSent as u32);
-const MULTI_STATE_FLAGS: u32 = (BOTH_SIDES_SHUTDOWN_MASK | ChannelState::PeerDisconnected as u32 | ChannelState::MonitorUpdateFailed as u32);
+const BOTH_SIDES_SHUTDOWN_MASK: u32 = ChannelState::LocalShutdownSent as u32 | ChannelState::RemoteShutdownSent as u32;
+const MULTI_STATE_FLAGS: u32 = BOTH_SIDES_SHUTDOWN_MASK | ChannelState::PeerDisconnected as u32 | ChannelState::MonitorUpdateFailed as u32;
 
 const INITIAL_COMMITMENT_NUMBER: u64 = (1 << 48) - 1;
 
@@ -382,7 +382,7 @@ pub const COMMITMENT_TX_WEIGHT_PER_HTLC: u64 = 172;
 
 /// Maximmum `funding_satoshis` value, according to the BOLT #2 specification
 /// it's 2^24.
-pub const MAX_FUNDING_SATOSHIS: u64 = (1 << 24);
+pub const MAX_FUNDING_SATOSHIS: u64 = 1 << 24;
 
 /// Used to return a simple Error back to ChannelManager. Will get converted to a
 /// msgs::ErrorAction::SendErrorMessage or msgs::ErrorAction::IgnoreError as appropriate with our