clippy it

RustCrypto · Aug 13, 2024 · 397e783 · 397e783
1 parent 640adca
commit 397e783
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 23 deletions.
diff --git a/sm2/src/pke.rs b/sm2/src/pke.rs
@@ -92,15 +92,15 @@ impl<'a> EncodeValue for Cipher<'a> {
     fn value_len(&self) -> elliptic_curve::pkcs8::der::Result<Length> {
         UintRef::new(&self.x.to_be_bytes())?.encoded_len()?
             + UintRef::new(&self.y.to_be_bytes())?.encoded_len()?
-            + OctetStringRef::new(&self.digest)?.encoded_len()?
-            + OctetStringRef::new(&self.cipher)?.encoded_len()?
+            + OctetStringRef::new(self.digest)?.encoded_len()?
+            + OctetStringRef::new(self.cipher)?.encoded_len()?
     }
 
     fn encode_value(&self, writer: &mut impl Writer) -> elliptic_curve::pkcs8::der::Result<()> {
         UintRef::new(&self.x.to_be_bytes())?.encode(writer)?;
         UintRef::new(&self.y.to_be_bytes())?.encode(writer)?;
-        OctetStringRef::new(&self.digest)?.encode(writer)?;
-        OctetStringRef::new(&self.cipher)?.encode(writer)?;
+        OctetStringRef::new(self.digest)?.encode(writer)?;
+        OctetStringRef::new(self.cipher)?.encode(writer)?;
         Ok(())
     }
 }
@@ -137,8 +137,8 @@ fn kdf(hasher: &mut dyn DynDigest, kpb: AffinePoint, c2: &mut [u8]) -> Result<()
     let encode_point = kpb.to_encoded_point(false);
 
     while offset < klen {
-        hasher.update(encode_point.x().unwrap());
-        hasher.update(encode_point.y().unwrap());
+        hasher.update(encode_point.x().ok_or(elliptic_curve::Error)?);
+        hasher.update(encode_point.y().ok_or(elliptic_curve::Error)?);
         hasher.update(&ct.to_be_bytes());
 
         hasher

diff --git a/sm2/src/pke/decrypting.rs b/sm2/src/pke/decrypting.rs
@@ -108,16 +108,16 @@ impl DecryptingKey {
         D: 'static + Digest + DynDigest + Send + Sync,
     {
         let cipher =
-            Cipher::from_der(&ciphertext).map_err(|e| elliptic_curve::pkcs8::Error::from(e))?;
+            Cipher::from_der(ciphertext).map_err(elliptic_curve::pkcs8::Error::from)?;
         let prefix: &[u8] = &[0x04];
         let x: [u8; 32] = cipher.x.to_be_bytes();
         let y: [u8; 32] = cipher.y.to_be_bytes();
-        let mut cipher = match self.mode {
+        let cipher = match self.mode {
             Mode::C1C2C3 => [prefix, &x, &y, cipher.cipher, cipher.digest].concat(),
             Mode::C1C3C2 => [prefix, &x, &y, cipher.digest, cipher.cipher].concat(),
         };
 
-        Ok(self.decrypt_digest::<D>(&mut cipher)?.to_vec())
+        Ok(self.decrypt_digest::<D>(&cipher)?.to_vec())
     }
 }
 
@@ -165,7 +165,7 @@ fn decrypt(
 
     // B1: get 𝐶1 from 𝐶
     let (c1, c) = cipher.split_at(c1_len as usize);
-    let encoded_c1 = EncodedPoint::from_bytes(c1).unwrap();
+    let encoded_c1 = EncodedPoint::from_bytes(c1).map_err(Error::from)?;
 
     // verify that point c1 satisfies the elliptic curve
     let mut c1_point = AffinePoint::from_encoded_point(&encoded_c1).unwrap();
@@ -195,9 +195,9 @@ fn decrypt(
     // compute 𝑢 = 𝐻𝑎𝑠ℎ(𝑥2 ∥ 𝑀′∥ 𝑦2).
     let mut u = vec![0u8; digest_size];
     let encode_point = c1_point.to_encoded_point(false);
-    hasher.update(&encode_point.x().unwrap());
-    hasher.update(&mut c2);
-    hasher.update(&encode_point.y().unwrap());
+    hasher.update(encode_point.x().ok_or(Error)?);
+    hasher.update(&c2);
+    hasher.update(encode_point.y().ok_or(Error)?);
     hasher.finalize_into_reset(&mut u).map_err(|_e| Error)?;
     let checked = u
         .iter()

diff --git a/sm2/src/pke/encrypting.rs b/sm2/src/pke/encrypting.rs
@@ -122,7 +122,7 @@ impl EncryptingKey {
             cipher,
         }
         .to_der()
-        .map_err(|e| elliptic_curve::pkcs8::Error::from(e))?)
+        .map_err(elliptic_curve::pkcs8::Error::from)?)
     }
 }
 
@@ -158,7 +158,7 @@ fn encrypt(
         }
 
         // A4: compute point [𝑘]𝑃𝐵 = (𝑥2, 𝑦2)
-        hpb = (s * &k).to_affine();
+        hpb = (s * k).to_affine();
 
         // A5: compute 𝑡 = 𝐾𝐷𝐹(𝑥2||𝑦2, 𝑘𝑙𝑒𝑛)
         // A6: compute 𝐶2 = 𝑀 ⊕ t
@@ -176,9 +176,9 @@ fn encrypt(
 
     // A7: compute 𝐶3 = 𝐻𝑎𝑠ℎ(𝑥2||𝑀||𝑦2)
     let mut c3 = vec![0; digest.output_size()];
-    digest.update(encode_point.x().unwrap());
+    digest.update(encode_point.x().ok_or(Error)?);
     digest.update(msg);
-    digest.update(encode_point.y().unwrap());
+    digest.update(encode_point.y().ok_or(Error)?);
     digest.finalize_into_reset(&mut c3).map_err(|_e| Error)?;
 
     // A8: output the ciphertext 𝐶 = 𝐶1||𝐶2||𝐶3.

diff --git a/sm2/tests/sm2pke.rs b/sm2/tests/sm2pke.rs
@@ -18,14 +18,14 @@ const ASN1_CIPHER: [u8; 116] = hex!("307202206ba17ad462a75beeb2caf8a1282687ab7e2
 
 #[test]
 fn decrypt_verify() {
-    let mut cipher = Vec::from(&CIPHER);
+    let cipher = Vec::from(&CIPHER);
     assert_eq!(
         DecryptingKey::new(
             NonZeroScalar::<Sm2>::try_from(PRIVATE_KEY.as_ref() as &[u8])
                 .unwrap()
                 .into()
         )
-        .decrypt(&mut cipher)
+        .decrypt(&cipher)
         .unwrap(),
         MSG
     );
@@ -73,16 +73,16 @@ proptest! {
     #[test]
     fn encrypt_and_decrpyt(dk in decrypting_key()) {
         let ek = dk.encrypting_key();
-        let mut cipher_bytes = ek.encrypt(MSG).unwrap();
-        assert_eq!(dk.decrypt(&mut cipher_bytes).unwrap(), MSG);
+        let cipher_bytes = ek.encrypt(MSG).unwrap();
+        assert_eq!(dk.decrypt(&cipher_bytes).unwrap(), MSG);
     }
 
     #[test]
     fn encrypt_and_decrpyt_mode(dk in decrypting_key_c1c2c3()) {
         let ek = dk.encrypting_key();
-        let mut cipher_bytes = ek.encrypt(MSG).unwrap();
+        let cipher_bytes = ek.encrypt(MSG).unwrap();
         assert_eq!(
-            dk.decrypt(&mut cipher_bytes)
+            dk.decrypt(&cipher_bytes)
                 .unwrap(),
             MSG
         );