diff --git a/src/mpris/player.rs b/src/mpris/player.rs
index a19a46e..0c4460d 100644
--- a/src/mpris/player.rs
+++ b/src/mpris/player.rs
@@ -333,7 +333,7 @@ impl Player {
     // FIXME: https://github.com/dbus2/zbus/issues/992
     #[zbus(property)]
     fn shuffle(&self) -> zbus::fdo::Result<bool> {
-        Err(zbus::fdo::Error::NotSupported("Shuffle".into()).into())
+        Err(zbus::fdo::Error::NotSupported("Shuffle".into()))
     }
 
     #[zbus(property)]
@@ -356,7 +356,8 @@ impl Player {
     #[tracing::instrument(skip(self), parent = None, target = "audrey::mpris", level = Level::DEBUG, ret)]
     fn set_volume(&self, volume: f64) -> zbus::Result<()> {
         let corrected_volume = (volume.clamp(0.0, 1.0) * 100.0) as i64;
-        Ok(self.window().set_volume(corrected_volume))
+        self.window().set_volume(corrected_volume);
+        Ok(())
     }
 
     #[zbus(property(emits_changed_signal = "false"))]