timedatectl: drop ifindex output parameter too

diff --git a/src/timedate/timedatectl.c b/src/timedate/timedatectl.c
index d51e29de1e6aa2742d4be678d6d4cb31b3a99bec..7d0201e6ff52785c3b4661069762f7d3cafd4a75 100644 (file)
--- a/src/timedate/timedatectl.c
+++ b/src/timedate/timedatectl.c
@@ -829,7 +829,7 @@ static int show_timesync(int argc, char **argv, void *userdata) {
         return 0;
 }
 
-static int parse_ifindex_bus(sd_bus *bus, const char *str, int *ret) {
+static int parse_ifindex_bus(sd_bus *bus, const char *str) {
         _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL;
         _cleanup_(sd_bus_message_unrefp) sd_bus_message *reply = NULL;
         int32_t i;
@@ -837,14 +837,11 @@ static int parse_ifindex_bus(sd_bus *bus, const char *str, int *ret) {
 
         assert(bus);
         assert(str);
-        assert(ret);
 
         r = parse_ifindex(str);
-        if (r >= 0) {
-                assert(r != 0);
-                *ret = r;
-                return 0;
-        }
+        if (r > 0)
+                return r;
+        assert(r < 0);
 
         r = sd_bus_call_method(
                         bus,
@@ -862,8 +859,7 @@ static int parse_ifindex_bus(sd_bus *bus, const char *str, int *ret) {
         if (r < 0)
                 return bus_log_create_error(r);
 
-        *ret = i;
-        return 0;
+        return i;
 }
 
 static int verb_ntp_servers(int argc, char **argv, void *userdata) {
@@ -874,9 +870,9 @@ static int verb_ntp_servers(int argc, char **argv, void *userdata) {
 
         assert(bus);
 
-        r = parse_ifindex_bus(bus, argv[1], &ifindex);
-        if (r < 0)
-                return r;
+        ifindex = parse_ifindex_bus(bus, argv[1]);
+        if (ifindex < 0)
+                return ifindex;
 
         polkit_agent_open_if_enabled(arg_transport, arg_ask_password);
 
@@ -912,9 +908,9 @@ static int verb_revert(int argc, char **argv, void *userdata) {
 
         assert(bus);
 
-        r = parse_ifindex_bus(bus, argv[1], &ifindex);
-        if (r < 0)
-                return r;
+        ifindex = parse_ifindex_bus(bus, argv[1]);
+        if (ifindex < 0)
+                return ifindex;
 
         polkit_agent_open_if_enabled(arg_transport, arg_ask_password);