fix cargo test-all-features

Author: jowparks

Cargo.toml

@@ -15,17 +15,17 @@ chacha20poly1305 = "0.10.1"
 ed25519-dalek = { version = "2.1.0", features = ["rand_core"] }
 rand_chacha = { version = "0.3.1", optional = true }
 rand_core = "0.6.4"
-reddsa = { git = "https://github.com/ZcashFoundation/reddsa.git", rev = "311baf8865f6e21527d1f20750d8f2cf5c9e531a", features = ["frost", "frost-rerandomized"] }
-siphasher = { version = "1.0.0", features =["serde_no_std"] }
+reddsa = { git = "https://github.com/ZcashFoundation/reddsa.git", rev = "b9c3107e6ec5333a89a7fa064f2d10f749a90cce", features = ["frost", "frost-rerandomized"] }
+siphasher = { version = "1.0.0" }
 x25519-dalek = { version = "2.0.0", features = ["reusable_secrets", "static_secrets"] }
 
 [dev-dependencies]
 hex-literal = "0.4.1"
 rand = "0.8.5"
 
 [features]
-default = ["signing"]
+default = ["signing", "dkg"]
 
 std = []
 signing = ["dep:blake3", "dep:rand_chacha", "std"]
-dkg = []
+dkg = ["signing"]

src/checksum.rs

@@ -30,8 +30,3 @@ impl fmt::Display for ChecksumError {
         }
     }
 }
-
-#[cfg(feature = "std")]
-use std::error;
-#[cfg(feature = "std")]
-impl error::Error for ChecksumError {}

src/dkg/round2.rs

@@ -329,18 +329,19 @@ impl CombinedPublicPackage {
         Ok(write_variable_length(
             writer,
             &self.packages,
-            |writer, pkg| Ok(pkg.serialize_without_sender_into(writer)?),
+            |writer, pkg| {
+                pkg.serialize_without_sender_into(writer)
+                    .map_err(|_| io::Error::other("serialize_into failed"))
+            },
         )?)
     }
 
-    pub fn deserialize_from<R: io::Read>(mut reader: R) -> io::Result<Self> {
+    pub fn deserialize_from<R: io::Read>(mut reader: R) -> Result<Self, IronfishFrostError> {
         let sender_identity = Identity::deserialize_from(&mut reader)?;
 
         let packages = read_variable_length(reader, move |reader| {
-            Ok(PublicPackage::deserialize_without_sender_from(
-                reader,
-                sender_identity.clone(),
-            )?)
+            PublicPackage::deserialize_without_sender_from(reader, sender_identity.clone())
+                .map_err(|_| io::Error::other("deserialization failed"))
         })?;
 
         Ok(Self { packages })

src/error.rs

@@ -6,12 +6,13 @@ use reddsa::frost::redjubjub::frost::Error as FrostError;
 use reddsa::frost::redjubjub::JubjubBlake2b512;
 
 use crate::checksum::ChecksumError;
+use crate::io;
 
 #[derive(Debug)]
 pub enum IronfishFrostError {
     InvalidInput,
     StdError,
-    IoError(std::io::Error),
+    IoError(io::Error),
     FrostError(FrostError<JubjubBlake2b512>),
     SignatureError(ed25519_dalek::SignatureError),
     ChecksumError(ChecksumError),
@@ -23,8 +24,8 @@ impl From<FrostError<JubjubBlake2b512>> for IronfishFrostError {
     }
 }
 
-impl From<std::io::Error> for IronfishFrostError {
-    fn from(error: std::io::Error) -> Self {
+impl From<io::Error> for IronfishFrostError {
+    fn from(error: io::Error) -> Self {
         IronfishFrostError::IoError(error)
     }
 }
@@ -34,12 +35,3 @@ impl From<ed25519_dalek::SignatureError> for IronfishFrostError {
         IronfishFrostError::SignatureError(error)
     }
 }
-
-impl From<IronfishFrostError> for std::io::Error {
-    fn from(error: IronfishFrostError) -> Self {
-        match error {
-            IronfishFrostError::IoError(e) => e,
-            _ => std::io::Error::new(std::io::ErrorKind::Other, format!("{:?}", error)),
-        }
-    }
-}