more detailed messages

src/cli.rs

@@ -209,13 +209,20 @@ pub fn run_cli() -> Fido2LuksResult<()> {
             }?;
             let added_slot = luks::add_key(device.clone(), &secret, &old_secret[..], Some(10))?;
             if *exclusive {
-                luks::remove_keyslots(&device, &[added_slot])?;
-            }
+                let destroyed = luks::remove_keyslots(&device, &[added_slot])?;
+                println!(
+                    "Added to key to device {}, slot: {}\nRemoved {} old keys",
+                    device.display(),
+                    added_slot,
+                    destroyed
+                );
+            } else {
                 println!(
                     "Added to key to device {}, slot: {}",
                     device.display(),
                     added_slot
                 );
+            }
             Ok(())
         }
         Command::ReplaceKey {

src/error.rs

@@ -58,7 +58,12 @@ impl From<FidoError> for Fido2LuksError {
 
 impl From<LibcryptErr> for Fido2LuksError {
     fn from(e: LibcryptErr) -> Self {
-        LuksError { cause: e }
+        match e {
+            LibcryptErr::IOError(e) if e.raw_os_error().iter().any(|code| code == &1i32) => {
+                WrongSecret
+            }
+            _ => LuksError { cause: e },
+        }
     }
 }
 impl From<io::Error> for Fido2LuksError {

src/luks.rs

@@ -1,9 +1,6 @@
 use crate::error::*;
 
-use libcryptsetup_rs::{
-    CryptActivateFlags, CryptDevice, CryptInit, CryptKeyslot, CryptVolumeKeyFlags,
-    EncryptionFormat, KeyslotInfo, LibcryptErr,
-};
+use libcryptsetup_rs::{CryptActivateFlags, CryptDevice, CryptInit, EncryptionFormat, KeyslotInfo};
 use std::path::Path;
 
 fn load_device_handle<P: AsRef<Path>>(path: P) -> Fido2LuksResult<CryptDevice> {
@@ -14,7 +11,7 @@ fn load_device_handle<P: AsRef<Path>>(path: P) -> Fido2LuksResult<CryptDevice> {
         .into_iter()
         .fold(None, |res, format| match res {
             Some(Ok(())) => res,
-            Some(e) => Some(e.and(load(format))),
+            Some(e) => Some(e.or(load(format))),
             None => Some(load(format)),
         })
         .unwrap()?;
@@ -47,16 +44,18 @@ pub fn add_key<P: AsRef<Path>>(
     Ok(slot)
 }
 
-pub fn remove_keyslots<P: AsRef<Path>>(path: P, exclude: &[u32]) -> Fido2LuksResult<()> {
+pub fn remove_keyslots<P: AsRef<Path>>(path: P, exclude: &[u32]) -> Fido2LuksResult<u32> {
     let mut device = load_device_handle(path)?;
     let mut slot = 0;
     let mut handle;
+    let mut destroyed = 0;
     loop {
         handle = device.keyslot_handle(Some(slot));
         match handle.status()? {
             KeyslotInfo::Inactive => continue,
             KeyslotInfo::Active if !exclude.contains(&slot) => {
-                dbg!((slot, handle.destroy()?));
+                handle.destroy()?;
+                destroyed += 1;
             }
             _ => (),
         }
@@ -66,7 +65,7 @@ pub fn remove_keyslots<P: AsRef<Path>>(path: P, exclude: &[u32]) -> Fido2LuksRes
         }
         slot += 1;
     }
-    Ok(())
+    Ok(destroyed)
 }
 
 pub fn replace_key<P: AsRef<Path>>(