diff --git a/openpgp-card-sequoia/src/card.rs b/openpgp-card-sequoia/src/card.rs index 02c5601..d08d779 100644 --- a/openpgp-card-sequoia/src/card.rs +++ b/openpgp-card-sequoia/src/card.rs @@ -76,7 +76,7 @@ impl<'a> Open<'a> { } pub fn verify_user(&mut self, pin: &[u8]) -> Result<(), Error> { - let _ = self.opt.verify_pw1_user(pin)?; + self.opt.verify_pw1_user(pin)?; self.pw1 = true; Ok(()) } @@ -84,13 +84,13 @@ impl<'a> Open<'a> { pub fn verify_user_pinpad(&mut self, pinpad_prompt: &dyn Fn()) -> Result<(), Error> { pinpad_prompt(); - let _ = self.opt.verify_pw1_user_pinpad()?; + self.opt.verify_pw1_user_pinpad()?; self.pw1 = true; Ok(()) } pub fn verify_user_for_signing(&mut self, pin: &[u8]) -> Result<(), Error> { - let _ = self.opt.verify_pw1_sign(pin)?; + self.opt.verify_pw1_sign(pin)?; // FIXME: depending on card mode, pw1_sign is only usable once @@ -104,7 +104,7 @@ impl<'a> Open<'a> { ) -> Result<(), Error> { pinpad_prompt(); - let _ = self.opt.verify_pw1_sign_pinpad()?; + self.opt.verify_pw1_sign_pinpad()?; // FIXME: depending on card mode, pw1_sign is only usable once @@ -113,7 +113,7 @@ impl<'a> Open<'a> { } pub fn verify_admin(&mut self, pin: &[u8]) -> Result<(), Error> { - let _ = self.opt.verify_pw3(pin)?; + self.opt.verify_pw3(pin)?; self.pw3 = true; Ok(()) } @@ -121,7 +121,7 @@ impl<'a> Open<'a> { pub fn verify_admin_pinpad(&mut self, pinpad_prompt: &dyn Fn()) -> Result<(), Error> { pinpad_prompt(); - let _ = self.opt.verify_pw3_pinpad()?; + self.opt.verify_pw3_pinpad()?; self.pw3 = true; Ok(()) } diff --git a/openpgp-card/src/openpgp.rs b/openpgp-card/src/openpgp.rs index 9850d14..d92fd81 100644 --- a/openpgp-card/src/openpgp.rs +++ b/openpgp-card/src/openpgp.rs @@ -254,7 +254,7 @@ impl<'a> OpenPgpTransaction<'a> { // Possible response data (Control Parameter = CP) don't need to be evaluated by the // application (See "7.2.5 SELECT DATA") - let _ = apdu::send_command(self.tx(), cmd, true)?.try_into()?; + apdu::send_command(self.tx(), cmd, true)?.try_into()?; Ok(()) }