Report errors from clock_gettime and clock_settime

src/slack-timedate.cpp

@@ -53,18 +53,20 @@ static void slack_method_call(const Glib::RefPtr<Gio::DBus::Connection>& connect
     {
         Glib::Variant<gint64> usec_utc;
         Glib::Variant<bool> relative;
+        Glib::Variant<bool> user_interaction;
 
         parameters.get_child(usec_utc, 0);
         parameters.get_child(relative, 1);
+        parameters.get_child(user_interaction, 2);
 
-        if (slack_set_time(usec_utc.get(), relative.get()))
+        try
         {
+            dlackware::timedate::set_time(usec_utc.get(), relative.get(), user_interaction.get());
             invocation->return_value(Glib::VariantContainerBase());
         }
-        else
+        catch (const std::system_error& error)
         {
-            Gio::DBus::Error error{ Gio::DBus::Error::FAILED, "Failed to set system clock" };
+            invocation->return_error(Gio::DBus::Error{ Gio::DBus::Error::FAILED, error.what() });
-            invocation->return_error(error);
         }
     }
     else if (method_name == "SetNTP")
@@ -293,9 +295,8 @@ namespace dlackware::timedate
     {
         return Glib::file_test("/etc/rc.d/rc.ntpd", Glib::FileTest::FILE_TEST_IS_EXECUTABLE);
     }
-}
 
-gboolean slack_set_time(gint64 seconds_since_epoch, gboolean relative)
+    void set_time(gint64 seconds_since_epoch, bool relative, bool)
     {
         timespec ts;
 
@@ -303,7 +304,7 @@ gboolean slack_set_time(gint64 seconds_since_epoch, gboolean relative)
         {
             if (clock_gettime(CLOCK_REALTIME, &ts) == -1)
             {
-            return FALSE;
+                throw std::system_error(errno, std::generic_category());
             }
             ts.tv_sec += static_cast<time_t>(seconds_since_epoch / dlackware::timedate::usec_per_sec);
             ts.tv_nsec += (seconds_since_epoch % dlackware::timedate::usec_per_sec) * dlackware::timedate::nsec_per_usec;
@@ -319,5 +320,9 @@ gboolean slack_set_time(gint64 seconds_since_epoch, gboolean relative)
             ts.tv_sec = static_cast<time_t>(seconds_since_epoch / dlackware::timedate::usec_per_sec);
             ts.tv_nsec = (seconds_since_epoch % dlackware::timedate::usec_per_sec) * dlackware::timedate::nsec_per_usec;
         }
-    return clock_settime (CLOCK_REALTIME, &ts) == 0;
+        if (clock_settime(CLOCK_REALTIME, &ts) == -1)
+        {
+            throw std::system_error(errno, std::generic_category());
+        }
+    }
 }

src/slack-timedate.h

@@ -47,12 +47,18 @@ namespace dlackware::timedate
     std::vector<Glib::ustring> list_timezones();
 
     // Sets the system time zone to the one passed by the argument
-    // Returns true on success, false otherwise
+    // Throws std::filesystem::filesystem_error.
     void set_timezone(const Glib::ustring& zone, bool user_interaction);
 
     // Sets NTP
+    // Throws std::filesystem::filesystem_error.
     void set_ntp(bool use_ntp, bool user_interaction);
 
+    // Changes the date/time
+    // Takes the amount of seconds since UNIX epoche and 
+    // Throws std::system_error.
+    void set_time(gint64 seconds_since_epoch, bool relative, bool user_interaction);
+
     class timedate1
     {
         const Gio::DBus::InterfaceVTable interface_vtable;
@@ -64,8 +70,3 @@ namespace dlackware::timedate
         void on_name_lost(const Glib::RefPtr<Gio::DBus::Connection>& connection, const Glib::ustring& name);
     };
 }
-
-// Changes the date/time
-// Takes the amount of seconds since UNIX epoche and 
-// Returns true on success, false otherwise
-gboolean slack_set_time(gint64 seconds_since_epoch, gboolean relative);