X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=src%2Fln%2Fpeer_channel_encryptor.rs;h=7db33c92e47288f77d9fe12b0b43662b78600f8a;hb=7b4f382e26368be7f010a081e553803b98eb9550;hp=2de38e68f8a0dd7ff297cd2e63a23fc43dde559c;hpb=f9fc2dfbe20f86ab8bc3e637f3ca33a5a868cb98;p=rust-lightning diff --git a/src/ln/peer_channel_encryptor.rs b/src/ln/peer_channel_encryptor.rs index 2de38e68..7db33c92 100644 --- a/src/ln/peer_channel_encryptor.rs +++ b/src/ln/peer_channel_encryptor.rs @@ -5,16 +5,14 @@ use secp256k1::Secp256k1; use secp256k1::key::{PublicKey,SecretKey}; use secp256k1::ecdh::SharedSecret; -use rand::{thread_rng,Rng}; - use crypto::digest::Digest; use crypto::hkdf::{hkdf_extract,hkdf_expand}; -use crypto::sha2::Sha256; use crypto::aead::{AeadEncryptor, AeadDecryptor}; use util::chacha20poly1305rfc::ChaCha20Poly1305RFC; -use util::byte_utils; +use util::{byte_utils,rng}; +use util::sha2::Sha256; // Sha256("Noise_XK_secp256k1_ChaChaPoly_SHA256") const NOISE_CK: [u8; 32] = [0x26, 0x40, 0xf5, 0x2e, 0xeb, 0xcd, 0x9e, 0x88, 0x29, 0x58, 0x95, 0x1c, 0x79, 0x42, 0x50, 0xee, 0xdb, 0x28, 0x00, 0x2c, 0x05, 0xd7, 0xdc, 0x2e, 0xa0, 0xf1, 0x95, 0x40, 0x60, 0x42, 0xca, 0xf1]; @@ -75,9 +73,8 @@ pub struct PeerChannelEncryptor { impl PeerChannelEncryptor { pub fn new_outbound(their_node_id: PublicKey) -> PeerChannelEncryptor { - let mut rng = thread_rng(); let mut key = [0u8; 32]; - rng.fill_bytes(&mut key); + rng::fill_bytes(&mut key); let secp_ctx = Secp256k1::new(); let sec_key = SecretKey::from_slice(&secp_ctx, &key).unwrap(); //TODO: nicer rng-is-bad error message @@ -150,19 +147,18 @@ impl PeerChannelEncryptor { let mut chacha = ChaCha20Poly1305RFC::new(key, &nonce, h); if !chacha.decrypt(&cyphertext[0..cyphertext.len() - 16], res, &cyphertext[cyphertext.len() - 16..]) { - return Err(HandleError{err: "Bad MAC", msg: Some(msgs::ErrorMessage::DisconnectPeer{})}); + return Err(HandleError{err: "Bad MAC", action: Some(msgs::ErrorAction::DisconnectPeer{})}); } Ok(()) } #[inline] fn hkdf(state: &mut BidirectionalNoiseState, ss: SharedSecret) -> [u8; 32] { - let sha = Sha256::new(); let mut hkdf = [0; 64]; { let mut prk = [0; 32]; - hkdf_extract(sha, &state.ck, &ss[..], &mut prk); - hkdf_expand(sha, &prk, &[0;0], &mut hkdf); + hkdf_extract(Sha256::new(), &state.ck, &ss[..], &mut prk); + hkdf_expand(Sha256::new(), &prk, &[0;0], &mut hkdf); } state.ck.copy_from_slice(&hkdf[0..32]); let mut res = [0; 32]; @@ -199,11 +195,11 @@ impl PeerChannelEncryptor { assert_eq!(act.len(), 50); if act[0] != 0 { - return Err(HandleError{err: "Unknown handshake version number", msg: Some(msgs::ErrorMessage::DisconnectPeer{})}); + return Err(HandleError{err: "Unknown handshake version number", action: Some(msgs::ErrorAction::DisconnectPeer{})}); } let their_pub = match PublicKey::from_slice(secp_ctx, &act[1..34]) { - Err(_) => return Err(HandleError{err: "Invalid public key", msg: Some(msgs::ErrorMessage::DisconnectPeer{})}), + Err(_) => return Err(HandleError{err: "Invalid public key", action: Some(msgs::ErrorAction::DisconnectPeer{})}), Ok(key) => key, }; @@ -276,9 +272,8 @@ impl PeerChannelEncryptor { pub fn process_act_one_with_key(&mut self, act_one: &[u8], our_node_secret: &SecretKey) -> Result<[u8; 50], HandleError> { assert_eq!(act_one.len(), 50); - let mut rng = thread_rng(); let mut key = [0u8; 32]; - rng.fill_bytes(&mut key); + rng::fill_bytes(&mut key); let our_ephemeral_key = SecretKey::from_slice(&self.secp_ctx, &key).unwrap(); //TODO: nicer rng-is-bad error message self.process_act_one_with_ephemeral_key(act_one, our_node_secret, our_ephemeral_key) } @@ -313,10 +308,9 @@ impl PeerChannelEncryptor { PeerChannelEncryptor::encrypt_with_ad(&mut res[50..], 0, &temp_k, &bidirectional_state.h, &[0; 0]); - sha.reset(); let mut prk = [0; 32]; - hkdf_extract(sha, &bidirectional_state.ck, &[0; 0], &mut prk); - hkdf_expand(sha, &prk, &[0;0], &mut final_hkdf); + hkdf_extract(Sha256::new(), &bidirectional_state.ck, &[0; 0], &mut prk); + hkdf_expand(Sha256::new(), &prk, &[0;0], &mut final_hkdf); ck = bidirectional_state.ck.clone(); res }, @@ -355,14 +349,14 @@ impl PeerChannelEncryptor { panic!("Requested act at wrong step"); } if act_three[0] != 0 { - return Err(HandleError{err: "Unknown handshake version number", msg: Some(msgs::ErrorMessage::DisconnectPeer{})}); + return Err(HandleError{err: "Unknown handshake version number", action: Some(msgs::ErrorAction::DisconnectPeer{})}); } let mut their_node_id = [0; 33]; PeerChannelEncryptor::decrypt_with_ad(&mut their_node_id, 1, &temp_k2.unwrap(), &bidirectional_state.h, &act_three[1..50])?; self.their_node_id = Some(match PublicKey::from_slice(&self.secp_ctx, &their_node_id) { Ok(key) => key, - Err(_) => return Err(HandleError{err: "Bad node_id from peer", msg: Some(msgs::ErrorMessage::DisconnectPeer{})}), + Err(_) => return Err(HandleError{err: "Bad node_id from peer", action: Some(msgs::ErrorAction::DisconnectPeer{})}), }); let mut sha = Sha256::new(); @@ -375,10 +369,9 @@ impl PeerChannelEncryptor { PeerChannelEncryptor::decrypt_with_ad(&mut [0; 0], 0, &temp_k, &bidirectional_state.h, &act_three[50..])?; - sha.reset(); let mut prk = [0; 32]; - hkdf_extract(sha, &bidirectional_state.ck, &[0; 0], &mut prk); - hkdf_expand(sha, &prk, &[0;0], &mut final_hkdf); + hkdf_extract(Sha256::new(), &bidirectional_state.ck, &[0; 0], &mut prk); + hkdf_expand(Sha256::new(), &prk, &[0;0], &mut final_hkdf); ck = bidirectional_state.ck.clone(); }, _ => panic!("Wrong direction for act"), @@ -416,11 +409,10 @@ impl PeerChannelEncryptor { match self.noise_state { NoiseState::Finished { ref mut sk, ref mut sn, ref mut sck, rk: _, rn: _, rck: _ } => { if *sn >= 1000 { - let mut sha = Sha256::new(); let mut prk = [0; 32]; - hkdf_extract(sha, sck, sk, &mut prk); + hkdf_extract(Sha256::new(), sck, sk, &mut prk); let mut hkdf = [0; 64]; - hkdf_expand(sha, &prk, &[0;0], &mut hkdf); + hkdf_expand(Sha256::new(), &prk, &[0;0], &mut hkdf); sck[..].copy_from_slice(&hkdf[0..32]); sk[..].copy_from_slice(&hkdf[32..]); @@ -447,11 +439,10 @@ impl PeerChannelEncryptor { match self.noise_state { NoiseState::Finished { sk: _, sn: _, sck: _, ref mut rk, ref mut rn, ref mut rck } => { if *rn >= 1000 { - let mut sha = Sha256::new(); let mut prk = [0; 32]; - hkdf_extract(sha, rck, rk, &mut prk); + hkdf_extract(Sha256::new(), rck, rk, &mut prk); let mut hkdf = [0; 64]; - hkdf_expand(sha, &prk, &[0;0], &mut hkdf); + hkdf_expand(Sha256::new(), &prk, &[0;0], &mut hkdf); rck[..].copy_from_slice(&hkdf[0..32]); rk[..].copy_from_slice(&hkdf[32..]); @@ -752,7 +743,7 @@ mod tests { let res = outbound_peer.encrypt_message(&msg); assert_eq!(res.len(), 5 + 2*16 + 2); - let mut len_header = res[0..2+16].to_vec(); + let len_header = res[0..2+16].to_vec(); assert_eq!(inbound_peer.decrypt_length_header(&len_header[..]).unwrap() as usize, msg.len()); assert_eq!(inbound_peer.decrypt_message(&res[2+16..]).unwrap()[..], msg[..]);