X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;ds=sidebyside;f=fuzz%2Ffuzz_targets%2Fchannel_target.rs;h=27891e11aa9aad2e96783659158fbc4d81aa5eac;hb=refs%2Fheads%2F2018-07-fees;hp=3f680be4d4ec7541113245c15f5e0d62f9949d53;hpb=4a9d3a5c0ad0ecef1b97329fc08873e18d2ba7fd;p=rust-lightning

diff --git a/fuzz/fuzz_targets/channel_target.rs b/fuzz/fuzz_targets/channel_target.rs
index 3f680be4..27891e11 100644
--- a/fuzz/fuzz_targets/channel_target.rs
+++ b/fuzz/fuzz_targets/channel_target.rs
@@ -12,6 +12,7 @@ use lightning::ln::channelmanager::{HTLCFailReason, PendingForwardHTLCInfo};
 use lightning::ln::msgs;
 use lightning::ln::msgs::MsgDecodable;
 use lightning::chain::chaininterface::{FeeEstimator, ConfirmationTarget};
+use lightning::chain::transaction::OutPoint;
 use lightning::util::reset_rng_state;
 
 use secp256k1::key::{PublicKey, SecretKey};
@@ -79,10 +80,10 @@ struct FuzzEstimator<'a> {
 	input: &'a InputData<'a>,
 }
 impl<'a> FeeEstimator for FuzzEstimator<'a> {
-	fn get_est_sat_per_vbyte(&self, _: ConfirmationTarget) -> u64 {
+	fn get_est_sat_per_1000_weight(&self, _: ConfirmationTarget) -> u64 {
 		//TODO: We should actually be testing at least much more than 64k...
 		match self.input.get_slice(2) {
-			Some(slice) => slice_to_be16(slice) as u64,
+			Some(slice) => slice_to_be16(slice) as u64 * 250,
 			None => 0
 		}
 	}
@@ -117,6 +118,7 @@ pub fn do_test(data: &[u8]) {
 					msgs::DecodeError::UnknownRealmByte => return,
 					msgs::DecodeError::BadPublicKey => return,
 					msgs::DecodeError::BadSignature => return,
+					msgs::DecodeError::BadText => return,
 					msgs::DecodeError::ExtraAddressesPerType => return,
 					msgs::DecodeError::WrongLength => panic!("We picked the length..."),
 				}
@@ -137,6 +139,7 @@ pub fn do_test(data: &[u8]) {
 						msgs::DecodeError::UnknownRealmByte => return,
 						msgs::DecodeError::BadPublicKey => return,
 						msgs::DecodeError::BadSignature => return,
+						msgs::DecodeError::BadText => return,
 						msgs::DecodeError::ExtraAddressesPerType => return,
 						msgs::DecodeError::WrongLength => panic!("We picked the length..."),
 					}
@@ -196,9 +199,9 @@ pub fn do_test(data: &[u8]) {
 		return_err!(chan.accept_channel(&accept_chan));
 
 		tx.output.push(TxOut{ value: chan_value, script_pubkey: chan.get_funding_redeemscript().to_v0_p2wsh() });
-		let funding_output = (Sha256dHash::from_data(&serialize(&tx).unwrap()[..]), 0);
+		let funding_output = OutPoint::new(Sha256dHash::from_data(&serialize(&tx).unwrap()[..]), 0);
 
-		chan.get_outbound_funding_created(funding_output.0.clone(), funding_output.1).unwrap();
+		chan.get_outbound_funding_created(funding_output).unwrap();
 		let funding_signed = decode_msg!(msgs::FundingSigned, 32+64);
 		return_err!(chan.funding_signed(&funding_signed));
 		chan
@@ -215,11 +218,11 @@ pub fn do_test(data: &[u8]) {
 		chan.get_accept_channel().unwrap();
 
 		tx.output.push(TxOut{ value: open_chan.funding_satoshis, script_pubkey: chan.get_funding_redeemscript().to_v0_p2wsh() });
-		let funding_output = (Sha256dHash::from_data(&serialize(&tx).unwrap()[..]), 0);
+		let funding_output = OutPoint::new(Sha256dHash::from_data(&serialize(&tx).unwrap()[..]), 0);
 
 		let mut funding_created = decode_msg!(msgs::FundingCreated, 32+32+2+64);
-		funding_created.funding_txid = funding_output.0.clone();
-		funding_created.funding_output_index = funding_output.1;
+		funding_created.funding_txid = funding_output.txid.clone();
+		funding_created.funding_output_index = funding_output.index;
 		return_err!(chan.funding_created(&funding_created));
 		chan
 	};