From dc18e8f24c2598a5eb4c6c7970492a6d6c63bf03 Mon Sep 17 00:00:00 2001 From: Deirdre Connolly Date: Wed, 9 Oct 2019 22:16:13 -0400 Subject: [PATCH] Remove .expect()s for block and transaction, they might fail for writer reasons --- zebra-network/src/protocol/codec.rs | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/zebra-network/src/protocol/codec.rs b/zebra-network/src/protocol/codec.rs index 0196f2f2..25c6f8f8 100644 --- a/zebra-network/src/protocol/codec.rs +++ b/zebra-network/src/protocol/codec.rs @@ -211,9 +211,7 @@ impl Codec { ref block, } => { writer.write_u32::(version.0)?; - block - .zcash_serialize(&mut writer) - .expect("Blocks must serialize."); + block.zcash_serialize(&mut writer)? } GetBlocks { ref version, @@ -268,9 +266,7 @@ impl Codec { ref transaction, } => { writer.write_u32::(version.0)?; - transaction - .zcash_serialize(&mut writer) - .expect("Transactions must serialize."); + transaction.zcash_serialize(&mut writer)? } // Mempool => {} // FilterLoad => {}