Map Engine errors to DbusErrorEnum::ERROR instead of INTERNAL_ERROR

Best to just use ERROR. fixes stratis-storage#1181 Signed-off-by: Andy Grover <[email protected]>
agrover · Sep 25, 2018 · 700e75c · 700e75c
1 parent 174ab35
commit 700e75c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/dbus_api/util.rs b/src/dbus_api/util.rs
@@ -45,7 +45,7 @@ where
 /// D-Bus methods return.
 pub fn engine_to_dbus_err_tuple(err: &StratisError) -> (u16, String) {
     let error = match *err {
-        StratisError::Error(_) => DbusErrorEnum::INTERNAL_ERROR,
+        StratisError::Error(_) => DbusErrorEnum::ERROR,
         StratisError::Engine(ref e, _) => match *e {
             ErrorEnum::Error => DbusErrorEnum::ERROR,
             ErrorEnum::AlreadyExists => DbusErrorEnum::ALREADY_EXISTS,
@@ -60,7 +60,7 @@ pub fn engine_to_dbus_err_tuple(err: &StratisError) -> (u16, String) {
         | StratisError::Serde(_)
         | StratisError::DM(_)
         | StratisError::Dbus(_)
-        | StratisError::Udev(_) => DbusErrorEnum::INTERNAL_ERROR,
+        | StratisError::Udev(_) => DbusErrorEnum::ERROR,
     };
     let description = match *err {
         StratisError::DM(DmError::Core(ref err)) => err.to_string(),