Merge pull request #25 from Dione-Software/minor-revision

Updated dependencies. Removed useless zeroize.

.gitignore

@@ -1,5 +1,2 @@
 /target
-Cargo.lock
-/.idea/.gitignore
-/.idea/double-ratchet-2.iml
-/.idea/vcs.xml
+/.idea

Cargo.toml

@@ -6,7 +6,7 @@ homepage = "https://github.com/Dione-Software/double-ratchet-2"
 repository = "https://github.com/Dione-Software/double-ratchet-2"
 readme = "README.md"
 keywords = ["double-ratchet", "crypto", "cryptography", "signal"]
-version = "0.3.6"
+version = "0.3.7"
 edition = "2018"
 license = "MIT"
 
@@ -17,20 +17,18 @@ maintenance = { status = "actively-developed" }
 
 
 [dependencies]
-p256 = {version = "0.9", features = ["zeroize", "ecdh", "arithmetic", "pem", "jwk"]}
+p256 = {version = "0.10", features = ["ecdh", "arithmetic", "pem", "jwk"]}
 rand_core = {version = "0.6"}
 getrandom = {version = "0.2.3"}
-hkdf = "0.11.0"
-hmac = "0.11.0"
+hkdf = "0.12"
+hmac = "0.12"
 aes-gcm-siv = {version = "0.10.3"}
-ring-compat = {version = "0.4.0", optional = true, features = ["digest"]}
-sha2 = {version = "0.9.5", optional = true}
-serde = {version = "1.0.125", default-features = false, features = ["derive"]}
-serde_bytes = "0.11.5"
-bincode = "1.3.3"
+sha2 = {version = "0.10"}
+serde = {version = "1", default-features = false, features = ["derive"]}
+serde_bytes = "0.11"
+bincode = "1"
 hashbrown = {version = "0.12", features = ["serde"]}
 zeroize = {version = "1.3", features = ["zeroize_derive"]}
-const-oid = "0.7.0"
 
 [dev-dependencies]
 criterion = "0.3.4"
@@ -41,9 +39,6 @@ harness = false
 
 [profile.release]
 lto = true
-opt-level = 3
 
 [features]
-default = ["sha2"]
-ring = ["ring-compat/digest"]
 wasm = ["getrandom/js"]

src/dh.rs

@@ -8,27 +8,12 @@ use alloc::vec::Vec;
 use alloc::string::ToString;
 use p256::elliptic_curve::ecdh::diffie_hellman;
 
-use zeroize::Zeroize;
-
 #[derive(Clone)]
 pub struct DhKeyPair {
     pub private_key: SecretKey,
     pub public_key: PublicKey,
 }
 
-impl Drop for DhKeyPair {
-    fn drop(&mut self) {
-        self.private_key = SecretKey::random(&mut OsRng);
-        self.public_key = self.private_key.public_key();
-    }
-}
-
-impl Zeroize for DhKeyPair {
-    fn zeroize(&mut self) {
-        self.private_key = SecretKey::random(&mut OsRng);
-        self.public_key = self.private_key.public_key();
-    }
-}
 
 impl DhKeyPair {
     fn ex_public_key_bytes(&self) -> Vec<u8> {
@@ -38,7 +23,7 @@ impl DhKeyPair {
 
 impl PartialEq for DhKeyPair {
     fn eq(&self, other: &Self) -> bool {
-        if self.private_key.to_bytes() != other.private_key.to_bytes() {
+        if self.private_key.to_be_bytes() != other.private_key.to_be_bytes() {
             return false
         }
         if self.ex_public_key_bytes() != other.ex_public_key_bytes() {
@@ -51,7 +36,7 @@ impl PartialEq for DhKeyPair {
 impl Debug for DhKeyPair {
     fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
         f.debug_struct("DhKeyPair")
-            .field("private_key", &self.private_key.to_bytes())
+            .field("private_key", &self.private_key.to_be_bytes())
             .field("public_key", &self.ex_public_key_bytes())
             .finish()
     }
@@ -74,7 +59,7 @@ impl DhKeyPair {
     }
 
     pub fn key_agreement(&self, public_key: &PublicKey) -> SharedSecret {
-        diffie_hellman(self.private_key.to_secret_scalar(), public_key.as_affine())
+        diffie_hellman(self.private_key.to_nonzero_scalar(), public_key.as_affine())
     }
 }
 

src/kdf_chain.rs

@@ -1,9 +1,5 @@
-use hmac::{Hmac, Mac, NewMac};
+use hmac::{Hmac, Mac};
 
-#[cfg(feature = "ring")]
-use ring_compat::digest::Sha512;
-
-#[cfg(not(feature = "ring"))]
 use sha2::Sha512;
 
 use core::convert::TryInto;

src/kdf_root.rs

@@ -1,10 +1,7 @@
 
 use hkdf::Hkdf;
 
-#[cfg(feature = "ring")]
-use ring_compat::digest::Sha512;
 
-#[cfg(not(feature = "ring"))]
 use sha2::Sha512;
 
 use core::convert::TryInto;

src/lib.rs

@@ -119,7 +119,7 @@
 //! # let shared_nhkb = [2; 32];
 //! let (bob_ratchet, public_key) = RatchetEncHeader::init_bob(sk, shared_hka, shared_nhkb);
 //! let ex_ratchet = bob_ratchet.export();
-//! let im_ratchet = RatchetEncHeader::import(&ex_ratchet);
+//! let im_ratchet = RatchetEncHeader::import(&ex_ratchet).unwrap();
 //! assert_eq!(im_ratchet, bob_ratchet)
 //! ```
 //!

src/ratchet.rs

@@ -33,7 +33,6 @@ pub struct Ratchet {
 
 impl Drop for Ratchet {
     fn drop(&mut self) {
-        self.dhs.zeroize();
         if let Some(mut _d) = self.dhr {
             let sk = SecretKey::random(&mut OsRng);
             _d = sk.public_key()
@@ -182,7 +181,6 @@ pub struct RatchetEncHeader {
 
 impl Zeroize for RatchetEncHeader {
     fn zeroize(&mut self) {
-        self.dhs.zeroize();
         self.rk.zeroize();
         self.cks.zeroize();
         self.ckr.zeroize();
@@ -225,7 +223,7 @@ impl From<&RatchetEncHeader> for ExRatchetEncHeader {
     fn from(reh: &RatchetEncHeader) -> Self {
         let private_dhs = reh.dhs.private_key.to_jwk_string();
         let public_dhs = reh.dhs.public_key.to_jwk_string();
-        let dhs = (private_dhs, public_dhs);
+        let dhs = (private_dhs.to_string(), public_dhs);
         let dhr = reh.dhr.map(|e| e.to_jwk_string());
         let rk = reh.rk;
         let cks = reh.cks;
@@ -441,8 +439,8 @@ impl RatchetEncHeader {
     }
 
     /// Import the ratchet from Binary data. Panics when binary data is invalid.
-    pub fn import(inp: &[u8]) -> Self {
-        let ex: ExRatchetEncHeader = bincode::deserialize(inp).unwrap();
-        RatchetEncHeader::from(&ex)
+    pub fn import(inp: &[u8]) -> Option<Self> {
+        let ex: ExRatchetEncHeader = bincode::deserialize(inp).ok()?;
+        Some(RatchetEncHeader::from(&ex))
     }
 }

tests/mod.rs

@@ -189,10 +189,10 @@ fn import_export() {
     let alice_ratchet = RatchetEncHeader::init_alice(sk, public_key, shared_hka, shared_nhkb);
 
     let ex_bob_ratchet = bob_ratchet.export();
-    let in_bob_ratchet = RatchetEncHeader::import(&ex_bob_ratchet);
+    let in_bob_ratchet = RatchetEncHeader::import(&ex_bob_ratchet).unwrap();
     assert_eq!(in_bob_ratchet, bob_ratchet);
 
     let ex_alice_ratchet = alice_ratchet.export();
-    let in_alice_ratchet = RatchetEncHeader::import(&ex_alice_ratchet);
+    let in_alice_ratchet = RatchetEncHeader::import(&ex_alice_ratchet).unwrap();
     assert_eq!(in_alice_ratchet, alice_ratchet);
 }