diff options
author | Lennart Poettering <lennart@poettering.net> | 2018-08-07 10:44:44 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-08-07 10:44:44 +0200 |
commit | 91f442401248fcb3676ba64707466d92a8643353 (patch) | |
tree | 8d3a64e0ebdfd6ff82ed77cfdb3610181352b4ae | |
parent | 6f663594bc5ee087cb15175cb691eaebcc45a0a1 (diff) | |
parent | fc95c359f6b6c179cbc235b42ddbc26485c0661b (diff) |
Merge pull request #9817 from yuwata/shorten-error-logging
tree-wide: Shorten error logging and several code cleanups
-rw-r--r-- | src/analyze/analyze.c | 30 | ||||
-rw-r--r-- | src/core/dbus-manager.c | 4 | ||||
-rw-r--r-- | src/core/mount.c | 3 | ||||
-rw-r--r-- | src/core/transaction.c | 18 | ||||
-rw-r--r-- | src/hostname/hostnamectl.c | 5 | ||||
-rw-r--r-- | src/initctl/initctl.c | 14 | ||||
-rw-r--r-- | src/libsystemd/sd-bus/test-bus-server.c | 6 | ||||
-rw-r--r-- | src/libsystemd/sd-device/sd-device.c | 6 | ||||
-rw-r--r-- | src/locale/localectl.c | 14 | ||||
-rw-r--r-- | src/locale/localed.c | 5 | ||||
-rw-r--r-- | src/login/loginctl.c | 75 | ||||
-rw-r--r-- | src/login/logind-action.c | 6 | ||||
-rw-r--r-- | src/login/logind-core.c | 4 | ||||
-rw-r--r-- | src/login/logind-session.c | 17 | ||||
-rw-r--r-- | src/login/logind-user.c | 20 | ||||
-rw-r--r-- | src/machine/machine.c | 35 | ||||
-rw-r--r-- | src/machine/machinectl.c | 78 | ||||
-rw-r--r-- | src/network/networkd-dhcp4.c | 6 | ||||
-rw-r--r-- | src/network/networkd-ipv4ll.c | 12 | ||||
-rw-r--r-- | src/network/networkd-link.c | 12 | ||||
-rw-r--r-- | src/nspawn/nspawn-register.c | 16 | ||||
-rw-r--r-- | src/resolve/resolvectl.c | 171 | ||||
-rw-r--r-- | src/run/run.c | 18 | ||||
-rw-r--r-- | src/shared/dissect-image.c | 6 | ||||
-rw-r--r-- | src/sulogin-shell/sulogin-shell.c | 4 | ||||
-rw-r--r-- | src/timedate/timedatectl.c | 16 |
26 files changed, 228 insertions, 373 deletions
diff --git a/src/analyze/analyze.c b/src/analyze/analyze.c index b723ebf9bd..6ecaca0163 100644 --- a/src/analyze/analyze.c +++ b/src/analyze/analyze.c @@ -162,10 +162,8 @@ static int bus_get_uint64_property(sd_bus *bus, const char *path, const char *in &error, 't', val); - if (r < 0) { - log_error("Failed to parse reply: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to parse reply: %s", bus_error_message(&error, -r)); return 0; } @@ -187,10 +185,8 @@ static int bus_get_unit_property_strv(sd_bus *bus, const char *path, const char property, &error, strv); - if (r < 0) { - log_error("Failed to get unit property %s: %s", property, bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get unit property %s: %s", property, bus_error_message(&error, -r)); return 0; } @@ -405,10 +401,8 @@ static int acquire_time_data(sd_bus *bus, struct unit_times **out) { "ListUnits", &error, &reply, NULL); - if (r < 0) { - log_error("Failed to list units: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to list units: %s", bus_error_message(&error, -r)); r = sd_bus_message_enter_container(reply, SD_BUS_TYPE_ARRAY, "(ssssssouso)"); if (r < 0) @@ -1031,10 +1025,8 @@ static int list_dependencies(sd_bus *bus, const char *name) { &error, &reply, "s"); - if (r < 0) { - log_error("Failed to get ID: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get ID: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "s", &id); if (r < 0) @@ -1293,10 +1285,8 @@ static int dot(int argc, char *argv[], void *userdata) { &error, &reply, ""); - if (r < 0) { - log_error("Failed to list units: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + log_error_errno(r, "Failed to list units: %s", bus_error_message(&error, -r)); r = sd_bus_message_enter_container(reply, SD_BUS_TYPE_ARRAY, "(ssssssouso)"); if (r < 0) diff --git a/src/core/dbus-manager.c b/src/core/dbus-manager.c index c2f82e19de..1d5237614e 100644 --- a/src/core/dbus-manager.c +++ b/src/core/dbus-manager.c @@ -1298,9 +1298,9 @@ int verify_run_space_and_log(const char *message) { r = verify_run_space(message, &error); if (r < 0) - log_error_errno(r, "%s", bus_error_message(&error, r)); + return log_error_errno(r, "%s", bus_error_message(&error, r)); - return r; + return 0; } static int method_reload(sd_bus_message *message, void *userdata, sd_bus_error *error) { diff --git a/src/core/mount.c b/src/core/mount.c index 55d851d8c0..b10bf1dae1 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -1584,8 +1584,7 @@ static int mount_setup_unit( return 0; fail: - log_warning_errno(r, "Failed to set up mount unit: %m"); - return r; + return log_warning_errno(r, "Failed to set up mount unit: %m"); } static int mount_load_proc_self_mountinfo(Manager *m, bool set_flags) { diff --git a/src/core/transaction.c b/src/core/transaction.c index 1c7efb207a..045930838b 100644 --- a/src/core/transaction.c +++ b/src/core/transaction.c @@ -695,10 +695,8 @@ int transaction_activate(Transaction *tr, Manager *m, JobMode mode, sd_bus_error if (r >= 0) break; - if (r != -EAGAIN) { - log_warning("Requested transaction contains an unfixable cyclic ordering dependency: %s", bus_error_message(e, r)); - return r; - } + if (r != -EAGAIN) + return log_warning_errno(r, "Requested transaction contains an unfixable cyclic ordering dependency: %s", bus_error_message(e, r)); /* Let's see if the resulting transaction ordering * graph is still cyclic... */ @@ -712,10 +710,8 @@ int transaction_activate(Transaction *tr, Manager *m, JobMode mode, sd_bus_error if (r >= 0) break; - if (r != -EAGAIN) { - log_warning("Requested transaction contains unmergeable jobs: %s", bus_error_message(e, r)); - return r; - } + if (r != -EAGAIN) + return log_warning_errno(r, "Requested transaction contains unmergeable jobs: %s", bus_error_message(e, r)); /* Seventh step: an entry got dropped, let's garbage * collect its dependencies. */ @@ -731,10 +727,8 @@ int transaction_activate(Transaction *tr, Manager *m, JobMode mode, sd_bus_error /* Ninth step: check whether we can actually apply this */ r = transaction_is_destructive(tr, mode, e); - if (r < 0) { - log_notice("Requested transaction contradicts existing jobs: %s", bus_error_message(e, r)); - return r; - } + if (r < 0) + return log_notice_errno(r, "Requested transaction contradicts existing jobs: %s", bus_error_message(e, r)); /* Tenth step: apply changes */ r = transaction_apply(tr, m, mode); diff --git a/src/hostname/hostnamectl.c b/src/hostname/hostnamectl.c index 8587f5c59f..fa4292c1fc 100644 --- a/src/hostname/hostnamectl.c +++ b/src/hostname/hostnamectl.c @@ -227,8 +227,9 @@ static int set_simple_string(sd_bus *bus, const char *method, const char *value) &error, NULL, "sb", value, arg_ask_password); if (r < 0) - log_error("Could not set property: %s", bus_error_message(&error, -r)); - return r; + return log_error_errno(r, "Could not set property: %s", bus_error_message(&error, -r)); + + return 0; } static int set_hostname(int argc, char **argv, void *userdata) { diff --git a/src/initctl/initctl.c b/src/initctl/initctl.c index 0c5aab6968..c93c02bf48 100644 --- a/src/initctl/initctl.c +++ b/src/initctl/initctl.c @@ -81,7 +81,7 @@ static const char *translate_runlevel(int runlevel, bool *isolate) { return NULL; } -static void change_runlevel(Server *s, int runlevel) { +static int change_runlevel(Server *s, int runlevel) { const char *target; _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; const char *mode; @@ -93,7 +93,7 @@ static void change_runlevel(Server *s, int runlevel) { target = translate_runlevel(runlevel, &isolate); if (!target) { log_warning("Got request for unknown runlevel %c, ignoring.", runlevel); - return; + return 0; } if (isolate) @@ -112,10 +112,10 @@ static void change_runlevel(Server *s, int runlevel) { &error, NULL, "ss", target, mode); - if (r < 0) { - log_error("Failed to change runlevel: %s", bus_error_message(&error, -r)); - return; - } + if (r < 0) + return log_error_errno(r, "Failed to change runlevel: %s", bus_error_message(&error, -r)); + + return 0; } static void request_process(Server *s, const struct init_request *req) { @@ -156,7 +156,7 @@ static void request_process(Server *s, const struct init_request *req) { break; default: - change_runlevel(s, req->runlevel); + (void) change_runlevel(s, req->runlevel); } return; diff --git a/src/libsystemd/sd-bus/test-bus-server.c b/src/libsystemd/sd-bus/test-bus-server.c index 17e46e5a2b..0f1b9645bc 100644 --- a/src/libsystemd/sd-bus/test-bus-server.c +++ b/src/libsystemd/sd-bus/test-bus-server.c @@ -128,10 +128,8 @@ static int client(struct context *c) { return log_error_errno(r, "Failed to allocate method call: %m"); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to issue method call: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to issue method call: %s", bus_error_message(&error, -r)); return 0; } diff --git a/src/libsystemd/sd-device/sd-device.c b/src/libsystemd/sd-device/sd-device.c index af29e74efa..0b810037a6 100644 --- a/src/libsystemd/sd-device/sd-device.c +++ b/src/libsystemd/sd-device/sd-device.c @@ -515,10 +515,8 @@ int device_read_uevent_file(sd_device *device) { else if (r == -ENOENT) /* some devices may not have uevent files, see set_syspath() */ return 0; - else if (r < 0) { - log_debug_errno(r, "sd-device: failed to read uevent file '%s': %m", path); - return r; - } + else if (r < 0) + return log_debug_errno(r, "sd-device: failed to read uevent file '%s': %m", path); for (i = 0; i < uevent_len; i++) switch (state) { diff --git a/src/locale/localectl.c b/src/locale/localectl.c index b3ad2820d9..ebc6a8ca8a 100644 --- a/src/locale/localectl.c +++ b/src/locale/localectl.c @@ -184,10 +184,8 @@ static int set_locale(int argc, char **argv, void *userdata) { return bus_log_create_error(r); r = sd_bus_call(bus, m, 0, &error, NULL); - if (r < 0) { - log_error("Failed to issue method call: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to issue method call: %s", bus_error_message(&error, -r)); return 0; } @@ -229,9 +227,9 @@ static int set_vconsole_keymap(int argc, char **argv, void *userdata) { NULL, "ssbb", map, toggle_map, arg_convert, arg_ask_password); if (r < 0) - log_error("Failed to set keymap: %s", bus_error_message(&error, -r)); + return log_error_errno(r, "Failed to set keymap: %s", bus_error_message(&error, -r)); - return r; + return 0; } static int list_vconsole_keymaps(int argc, char **argv, void *userdata) { @@ -273,9 +271,9 @@ static int set_x11_keymap(int argc, char **argv, void *userdata) { "ssssbb", layout, model, variant, options, arg_convert, arg_ask_password); if (r < 0) - log_error("Failed to set keymap: %s", bus_error_message(&error, -r)); + return log_error_errno(r, "Failed to set keymap: %s", bus_error_message(&error, -r)); - return r; + return 0; } static int list_x11_keymaps(int argc, char **argv, void *userdata) { diff --git a/src/locale/localed.c b/src/locale/localed.c index 71115c59d7..1f0c3ac74f 100644 --- a/src/locale/localed.c +++ b/src/locale/localed.c @@ -104,8 +104,9 @@ static int vconsole_reload(sd_bus *bus) { "ss", "systemd-vconsole-setup.service", "replace"); if (r < 0) - log_error("Failed to issue method call: %s", bus_error_message(&error, -r)); - return r; + return log_error_errno(r, "Failed to issue method call: %s", bus_error_message(&error, -r)); + + return 0; } static int vconsole_convert_to_x11_and_emit(Context *c, sd_bus_message *m) { diff --git a/src/login/loginctl.c b/src/login/loginctl.c index be55fdbfd8..9b3fed928b 100644 --- a/src/login/loginctl.c +++ b/src/login/loginctl.c @@ -856,10 +856,9 @@ static int show_session(int argc, char *argv[], void *userdata) { session = getenv("XDG_SESSION_ID"); if (session) { r = get_session_path(bus, session, &error, &path); - if (r < 0) { - log_error("Failed to get session path: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get session path: %s", bus_error_message(&error, r)); + p = path; } @@ -868,10 +867,8 @@ static int show_session(int argc, char *argv[], void *userdata) { for (i = 1; i < argc; i++) { r = get_session_path(bus, argv[i], &error, &path); - if (r < 0) { - log_error("Failed to get session path: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get session path: %s", bus_error_message(&error, r)); if (properties) r = show_properties(bus, path, &new_line); @@ -924,10 +921,8 @@ static int show_user(int argc, char *argv[], void *userdata) { "GetUser", &error, &reply, "u", (uint32_t) uid); - if (r < 0) { - log_error("Failed to get user: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get user: %s", bus_error_message(&error, r)); r = sd_bus_message_read(reply, "o", &path); if (r < 0) @@ -979,10 +974,8 @@ static int show_seat(int argc, char *argv[], void *userdata) { "GetSeat", &error, &reply, "s", argv[i]); - if (r < 0) { - log_error("Failed to get seat: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get seat: %s", bus_error_message(&error, r)); r = sd_bus_message_read(reply, "o", &path); if (r < 0) @@ -1036,10 +1029,8 @@ static int activate(int argc, char *argv[], void *userdata) { "ActivateSession", &error, NULL, "s", argv[i]); - if (r < 0) { - log_error("Failed to issue method call: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to issue method call: %s", bus_error_message(&error, -r)); } return 0; @@ -1068,10 +1059,8 @@ static int kill_session(int argc, char *argv[], void *userdata) { "KillSession", &error, NULL, "ssi", argv[i], arg_kill_who, arg_signal); - if (r < 0) { - log_error("Could not kill session: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not kill session: %s", bus_error_message(&error, -r)); } return 0; @@ -1121,10 +1110,8 @@ static int enable_linger(int argc, char *argv[], void *userdata) { "SetUserLinger", &error, NULL, "ubb", (uint32_t) uid, b, true); - if (r < 0) { - log_error("Could not enable linger: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not enable linger: %s", bus_error_message(&error, -r)); } return 0; @@ -1155,10 +1142,8 @@ static int terminate_user(int argc, char *argv[], void *userdata) { "TerminateUser", &error, NULL, "u", (uint32_t) uid); - if (r < 0) { - log_error("Could not terminate user: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not terminate user: %s", bus_error_message(&error, -r)); } return 0; @@ -1192,10 +1177,8 @@ static int kill_user(int argc, char *argv[], void *userdata) { "KillUser", &error, NULL, "ui", (uint32_t) uid, arg_signal); - if (r < 0) { - log_error("Could not kill user: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not kill user: %s", bus_error_message(&error, -r)); } return 0; @@ -1222,10 +1205,8 @@ static int attach(int argc, char *argv[], void *userdata) { &error, NULL, "ssb", argv[1], argv[i], true); - if (r < 0) { - log_error("Could not attach device: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not attach device: %s", bus_error_message(&error, -r)); } return 0; @@ -1250,9 +1231,9 @@ static int flush_devices(int argc, char *argv[], void *userdata) { &error, NULL, "b", true); if (r < 0) - log_error("Could not flush devices: %s", bus_error_message(&error, -r)); + return log_error_errno(r, "Could not flush devices: %s", bus_error_message(&error, -r)); - return r; + return 0; } static int lock_sessions(int argc, char *argv[], void *userdata) { @@ -1274,9 +1255,9 @@ static int lock_sessions(int argc, char *argv[], void *userdata) { &error, NULL, NULL); if (r < 0) - log_error("Could not lock sessions: %s", bus_error_message(&error, -r)); + return log_error_errno(r, "Could not lock sessions: %s", bus_error_message(&error, -r)); - return r; + return 0; } static int terminate_seat(int argc, char *argv[], void *userdata) { @@ -1299,10 +1280,8 @@ static int terminate_seat(int argc, char *argv[], void *userdata) { "TerminateSeat", &error, NULL, "s", argv[i]); - if (r < 0) { - log_error("Could not terminate seat: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not terminate seat: %s", bus_error_message(&error, -r)); } return 0; diff --git a/src/login/logind-action.c b/src/login/logind-action.c index 08e41af81a..317e9ef384 100644 --- a/src/login/logind-action.c +++ b/src/login/logind-action.c @@ -152,10 +152,8 @@ int manager_handle_action( log_info("%s", message_table[handle]); r = bus_manager_shutdown_or_sleep_now_or_later(m, target, inhibit_operation, &error); - if (r < 0) { - log_error("Failed to execute operation: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to execute operation: %s", bus_error_message(&error, r)); return 1; } diff --git a/src/login/logind-core.c b/src/login/logind-core.c index dbae4bf5af..d78d928d03 100644 --- a/src/login/logind-core.c +++ b/src/login/logind-core.c @@ -522,9 +522,9 @@ int manager_spawn_autovt(Manager *m, unsigned int vtnr) { NULL, "ss", name, "fail"); if (r < 0) - log_error("Failed to start %s: %s", name, bus_error_message(&error, r)); + return log_error_errno(r, "Failed to start %s: %s", name, bus_error_message(&error, r)); - return r; + return 0; } static bool manager_is_docked(Manager *m) { diff --git a/src/login/logind-session.c b/src/login/logind-session.c index 69d5a10319..226cc49b01 100644 --- a/src/login/logind-session.c +++ b/src/login/logind-session.c @@ -547,7 +547,8 @@ static int session_start_scope(Session *s, sd_bus_message *properties) { if (!s->scope) { _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; - char *scope, *job = NULL; + _cleanup_free_ char *scope = NULL; + char *job = NULL; const char *description; scope = strjoin("session-", s->id, ".scope"); @@ -567,16 +568,12 @@ static int session_start_scope(Session *s, sd_bus_message *properties) { properties, &error, &job); - if (r < 0) { - log_error_errno(r, "Failed to start session scope %s: %s", scope, bus_error_message(&error, r)); - free(scope); - return r; - } else { - s->scope = scope; + if (r < 0) + return log_error_errno(r, "Failed to start session scope %s: %s", scope, bus_error_message(&error, r)); - free(s->scope_job); - s->scope_job = job; - } + + s->scope = TAKE_PTR(scope); + free_and_replace(s->scope_job, job); } if (s->scope) diff --git a/src/login/logind-user.c b/src/login/logind-user.c index 346f792fb8..f2664c323e 100644 --- a/src/login/logind-user.c +++ b/src/login/logind-user.c @@ -391,15 +391,10 @@ static int user_stop_slice(User *u) { assert(u); r = manager_stop_unit(u->manager, u->slice, &error, &job); - if (r < 0) { - log_error("Failed to stop user slice: %s", bus_error_message(&error, r)); - return r; - } - - free(u->slice_job); - u->slice_job = job; + if (r < 0) + return log_error_errno(r, "Failed to stop user slice: %s", bus_error_message(&error, r)); - return r; + return free_and_replace(u->slice_job, job); } static int user_stop_service(User *u) { @@ -410,13 +405,10 @@ static int user_stop_service(User *u) { assert(u); r = manager_stop_unit(u->manager, u->service, &error, &job); - if (r < 0) { - log_error("Failed to stop user service: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to stop user service: %s", bus_error_message(&error, r)); - free_and_replace(u->service_job, job); - return r; + return free_and_replace(u->service_job, job); } int user_stop(User *u, bool force) { diff --git a/src/machine/machine.c b/src/machine/machine.c index b645f614c4..6ab33d3bea 100644 --- a/src/machine/machine.c +++ b/src/machine/machine.c @@ -328,14 +328,13 @@ int machine_load(Machine *m) { } static int machine_start_scope(Machine *m, sd_bus_message *properties, sd_bus_error *error) { - int r = 0; - assert(m); assert(m->class != MACHINE_HOST); if (!m->unit) { - _cleanup_free_ char *escaped = NULL; - char *scope, *description, *job = NULL; + _cleanup_free_ char *escaped = NULL, *scope = NULL; + char *description, *job = NULL; + int r; escaped = unit_name_escape(m->name); if (!escaped) @@ -348,22 +347,17 @@ static int machine_start_scope(Machine *m, sd_bus_message *properties, sd_bus_er description = strjoina(m->class == MACHINE_VM ? "Virtual Machine " : "Container ", m->name); r = manager_start_scope(m->manager, scope, m->leader, SPECIAL_MACHINE_SLICE, description, properties, error, &job); - if (r < 0) { - log_error("Failed to start machine scope: %s", bus_error_message(error, r)); - free(scope); - return r; - } else { - m->unit = scope; - - free(m->scope_job); - m->scope_job = job; - } + if (r < 0) + return log_error_errno(r, "Failed to start machine scope: %s", bus_error_message(error, r)); + + m->unit = TAKE_PTR(scope); + free_and_replace(m->scope_job, job); } if (m->unit) hashmap_put(m->manager->machine_units, m->unit, m); - return r; + return 0; } int machine_start(Machine *m, sd_bus_message *properties, sd_bus_error *error) { @@ -417,15 +411,10 @@ static int machine_stop_scope(Machine *m) { return 0; r = manager_stop_unit(m->manager, m->unit, &error, &job); - if (r < 0) { - log_error("Failed to stop machine scope: %s", bus_error_message(&error, r)); - return r; - } - - free(m->scope_job); - m->scope_job = job; + if (r < 0) + return log_error_errno(r, "Failed to stop machine scope: %s", bus_error_message(&error, r)); - return 0; + return free_and_replace(m->scope_job, job); } int machine_stop(Machine *m) { diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c index 9ba9ea97e7..f083dd2f81 100644 --- a/src/machine/machinectl.c +++ b/src/machine/machinectl.c @@ -773,10 +773,8 @@ static int show_machine(int argc, char *argv[], void *userdata) { &error, &reply, "s", argv[i]); - if (r < 0) { - log_error("Could not get path to machine: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not get path to machine: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "o", &path); if (r < 0) @@ -1118,10 +1116,8 @@ static int show_image(int argc, char *argv[], void *userdata) { &error, &reply, "s", argv[i]); - if (r < 0) { - log_error("Could not get path to image: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not get path to image: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "o", &path); if (r < 0) @@ -1158,10 +1154,8 @@ static int kill_machine(int argc, char *argv[], void *userdata) { &error, NULL, "ssi", argv[i], arg_kill_who, arg_signal); - if (r < 0) { - log_error("Could not kill machine: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not kill machine: %s", bus_error_message(&error, -r)); } return 0; @@ -1200,10 +1194,8 @@ static int terminate_machine(int argc, char *argv[], void *userdata) { &error, NULL, "s", argv[i]); - if (r < 0) { - log_error("Could not terminate machine: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not terminate machine: %s", bus_error_message(&error, -r)); } return 0; @@ -1285,10 +1277,8 @@ static int bind_mount(int argc, char *argv[], void *userdata) { argv[3], arg_read_only, arg_mkdir); - if (r < 0) { - log_error("Failed to bind mount: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to bind mount: %s", bus_error_message(&error, -r)); return 0; } @@ -1459,10 +1449,8 @@ static int login_machine(int argc, char *argv[], void *userdata) { &error, &reply, "s", machine); - if (r < 0) { - log_error("Failed to get login PTY: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get login PTY: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "hs", &master, NULL); if (r < 0) @@ -1613,10 +1601,8 @@ static int rename_image(int argc, char *argv[], void *userdata) { &error, NULL, "ss", argv[1], argv[2]); - if (r < 0) { - log_error("Could not rename image: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not rename image: %s", bus_error_message(&error, -r)); return 0; } @@ -1679,10 +1665,8 @@ static int read_only_image(int argc, char *argv[], void *userdata) { &error, NULL, "sb", argv[1], b); - if (r < 0) { - log_error("Could not mark image read-only: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not mark image read-only: %s", bus_error_message(&error, -r)); return 0; } @@ -1771,10 +1755,8 @@ static int start_machine(int argc, char *argv[], void *userdata) { &error, &reply, "ss", unit, "fail"); - if (r < 0) { - log_error("Failed to start unit: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to start unit: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "o", &object); if (r < 0) @@ -1853,10 +1835,8 @@ static int enable_machine(int argc, char *argv[], void *userdata) { return bus_log_create_error(r); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to enable or disable unit: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to enable or disable unit: %s", bus_error_message(&error, -r)); if (streq(argv[0], "enable")) { r = sd_bus_message_read(reply, "b", NULL); @@ -1991,10 +1971,8 @@ static int transfer_image_common(sd_bus *bus, sd_bus_message *m) { return log_error_errno(r, "Failed to request match: %m"); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to transfer image: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to transfer image: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "uo", &id, &path); if (r < 0) @@ -2428,10 +2406,8 @@ static int list_transfers(int argc, char *argv[], void *userdata) { &error, &reply, NULL); - if (r < 0) { - log_error("Could not get transfers: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not get transfers: %s", bus_error_message(&error, -r)); r = sd_bus_message_enter_container(reply, 'a', "(usssdo)"); if (r < 0) @@ -2526,10 +2502,8 @@ static int cancel_transfer(int argc, char *argv[], void *userdata) { &error, NULL, "u", id); - if (r < 0) { - log_error("Could not cancel transfer: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Could not cancel transfer: %s", bus_error_message(&error, -r)); } return 0; diff --git a/src/network/networkd-dhcp4.c b/src/network/networkd-dhcp4.c index 08656334e0..7798844e03 100644 --- a/src/network/networkd-dhcp4.c +++ b/src/network/networkd-dhcp4.c @@ -503,10 +503,8 @@ static int dhcp_lease_acquired(sd_dhcp_client *client, Link *link) { if (!link->network->dhcp_critical) { r = sd_dhcp_lease_get_lifetime(link->dhcp_lease, &lifetime); - if (r < 0) { - log_link_warning_errno(link, r, "DHCP error: no lifetime: %m"); - return r; - } + if (r < 0) + return log_link_warning_errno(link, r, "DHCP error: no lifetime: %m"); } r = dhcp4_update_address(link, &address, &netmask, lifetime); diff --git a/src/network/networkd-ipv4ll.c b/src/network/networkd-ipv4ll.c index a710240f94..66f775432f 100644 --- a/src/network/networkd-ipv4ll.c +++ b/src/network/networkd-ipv4ll.c @@ -26,10 +26,8 @@ static int ipv4ll_address_lost(Link *link) { log_link_debug(link, "IPv4 link-local release %u.%u.%u.%u", ADDRESS_FMT_VAL(addr)); r = address_new(&address); - if (r < 0) { - log_link_error_errno(link, r, "Could not allocate address: %m"); - return r; - } + if (r < 0) + return log_link_error_errno(link, r, "Could not allocate address: %m"); address->family = AF_INET; address->in_addr.in = addr; @@ -39,10 +37,8 @@ static int ipv4ll_address_lost(Link *link) { address_remove(address, link, link_address_remove_handler); r = route_new(&route); - if (r < 0) { - log_link_error_errno(link, r, "Could not allocate route: %m"); - return r; - } + if (r < 0) + return log_link_error_errno(link, r, "Could not allocate route: %m"); route->family = AF_INET; route->scope = RT_SCOPE_LINK; diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c index c603f4290a..4c534e685d 100644 --- a/src/network/networkd-link.c +++ b/src/network/networkd-link.c @@ -1175,10 +1175,8 @@ static int link_enter_set_addresses(Link *link) { } r = sd_dhcp_server_set_emit_router(link->dhcp_server, link->network->dhcp_server_emit_router); - if (r < 0) { - log_link_warning_errno(link, r, "Failed to set router emission for DHCP server: %m"); - return r; - } + if (r < 0) + return log_link_warning_errno(link, r, "Failed to set router emission for DHCP server: %m"); if (link->network->dhcp_server_emit_timezone) { _cleanup_free_ char *buffer = NULL; @@ -3386,10 +3384,8 @@ int link_update(Link *link, sd_netlink_message *m) { if (link->dhcp_client) { r = sd_dhcp_client_set_mtu(link->dhcp_client, link->mtu); - if (r < 0) { - log_link_warning_errno(link, r, "Could not update MTU in DHCP client: %m"); - return r; - } + if (r < 0) + return log_link_warning_errno(link, r, "Could not update MTU in DHCP client: %m"); } if (link->radv) { diff --git a/src/nspawn/nspawn-register.c b/src/nspawn/nspawn-register.c index 93185ecaaa..85f3cf1c01 100644 --- a/src/nspawn/nspawn-register.c +++ b/src/nspawn/nspawn-register.c @@ -195,10 +195,8 @@ int register_machine( r = sd_bus_call(bus, m, 0, &error, NULL); } - if (r < 0) { - log_error("Failed to register machine: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to register machine: %s", bus_error_message(&error, r)); return 0; } @@ -242,10 +240,8 @@ int terminate_machine(sd_bus *bus, pid_t pid) { &error, NULL, NULL); - if (r < 0) { + if (r < 0) log_debug("Failed to terminate machine: %s", bus_error_message(&error, r)); - return 0; - } return 0; } @@ -336,10 +332,8 @@ int allocate_scope( return bus_log_create_error(r); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to allocate scope: %s", bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to allocate scope: %s", bus_error_message(&error, r)); r = sd_bus_message_read(reply, "o", &object); if (r < 0) diff --git a/src/resolve/resolvectl.c b/src/resolve/resolvectl.c index 7f29546de1..f8372d5e83 100644 --- a/src/resolve/resolvectl.c +++ b/src/resolve/resolvectl.c @@ -324,10 +324,8 @@ static int resolve_address(sd_bus *bus, int family, const union in_addr_union *a ts = now(CLOCK_MONOTONIC); r = sd_bus_call(bus, req, SD_RESOLVED_QUERY_TIMEOUT_USEC, &error, &reply); - if (r < 0) { - log_error("%s: resolve call failed: %s", pretty, bus_error_message(&error, r)); - return r; - } + if (r < 0) + return log_error_errno(r, "%s: resolve call failed: %s", pretty, bus_error_message(&error, r)); ts = now(CLOCK_MONOTONIC) - ts; @@ -1370,21 +1368,27 @@ static int status_print_strv_ifindex(int ifindex, const char *ifname, char **p) return 0; } -static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode mode, bool *empty_line) { - - struct link_info { - uint64_t scopes_mask; - const char *llmnr; - const char *mdns; - const char *dns_over_tls; - const char *dnssec; - char *current_dns; - char **dns; - char **domains; - char **ntas; - bool dnssec_supported; - } link_info = {}; +struct link_info { + uint64_t scopes_mask; + const char *llmnr; + const char *mdns; + const char *dns_over_tls; + const char *dnssec; + char *current_dns; + char **dns; + char **domains; + char **ntas; + bool dnssec_supported; +}; + +static void link_info_clear(struct link_info *p) { + free(p->current_dns); + strv_free(p->dns); + strv_free(p->domains); + strv_free(p->ntas); +} +static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode mode, bool *empty_line) { static const struct bus_properties_map property_map[] = { { "ScopesMask", "t", NULL, offsetof(struct link_info, scopes_mask) }, { "DNS", "a(iay)", map_link_dns_servers, offsetof(struct link_info, dns) }, @@ -1398,9 +1402,9 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode { "DNSSECSupported", "b", NULL, offsetof(struct link_info, dnssec_supported) }, {} }; - _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_(sd_bus_message_unrefp) sd_bus_message *m = NULL; + _cleanup_(link_info_clear) struct link_info link_info = {}; _cleanup_free_ char *ifi = NULL, *p = NULL; char ifname[IF_NAMESIZE] = ""; char **i; @@ -1431,35 +1435,26 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode &error, &m, &link_info); - if (r < 0) { - log_error_errno(r, "Failed to get link data for %i: %s", ifindex, bus_error_message(&error, r)); - goto finish; - } + if (r < 0) + return log_error_errno(r, "Failed to get link data for %i: %s", ifindex, bus_error_message(&error, r)); (void) pager_open(arg_no_pager, false); - if (mode == STATUS_DNS) { - r = status_print_strv_ifindex(ifindex, name, link_info.dns); - goto finish; - } + if (mode == STATUS_DNS) + return status_print_strv_ifindex(ifindex, name, link_info.dns); - if (mode == STATUS_DOMAIN) { - r = status_print_strv_ifindex(ifindex, name, link_info.domains); - goto finish; - } + if (mode == STATUS_DOMAIN) + return status_print_strv_ifindex(ifindex, name, link_info.domains); - if (mode == STATUS_NTA) { - r = status_print_strv_ifindex(ifindex, name, link_info.ntas); - goto finish; - } + if (mode == STATUS_NTA) + return status_print_strv_ifindex(ifindex, name, link_info.ntas); if (mode == STATUS_LLMNR) { printf("%sLink %i (%s)%s: %s\n", ansi_highlight(), ifindex, name, ansi_normal(), strna(link_info.llmnr)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_MDNS) { @@ -1467,8 +1462,7 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode ansi_highlight(), ifindex, name, ansi_normal(), strna(link_info.mdns)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_PRIVATE) { @@ -1476,8 +1470,7 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode ansi_highlight(), ifindex, name, ansi_normal(), strna(link_info.dns_over_tls)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_DNSSEC) { @@ -1485,8 +1478,7 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode ansi_highlight(), ifindex, name, ansi_normal(), strna(link_info.dnssec)); - r = 0; - goto finish; + return 0; } if (empty_line && *empty_line) @@ -1540,14 +1532,7 @@ static int status_ifindex(sd_bus *bus, int ifindex, const char *name, StatusMode if (empty_line) *empty_line = true; - r = 0; - -finish: - free(link_info.current_dns); - strv_free(link_info.dns); - strv_free(link_info.domains); - strv_free(link_info.ntas); - return r; + return 0; } static int map_global_dns_servers(sd_bus *bus, const char *member, sd_bus_message *m, sd_bus_error *error, void *userdata) { @@ -1644,21 +1629,28 @@ static int status_print_strv_global(char **p) { return 0; } -static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { - - struct global_info { - char *current_dns; - char **dns; - char **fallback_dns; - char **domains; - char **ntas; - const char *llmnr; - const char *mdns; - const char *dns_over_tls; - const char *dnssec; - bool dnssec_supported; - } global_info = {}; +struct global_info { + char *current_dns; + char **dns; + char **fallback_dns; + char **domains; + char **ntas; + const char *llmnr; + const char *mdns; + const char *dns_over_tls; + const char *dnssec; + bool dnssec_supported; +}; + +static void global_info_clear(struct global_info *p) { + free(p->current_dns); + strv_free(p->dns); + strv_free(p->fallback_dns); + strv_free(p->domains); + strv_free(p->ntas); +} +static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { static const struct bus_properties_map property_map[] = { { "DNS", "a(iiay)", map_global_dns_servers, offsetof(struct global_info, dns) }, { "FallbackDNS", "a(iiay)", map_global_dns_servers, offsetof(struct global_info, fallback_dns) }, @@ -1672,9 +1664,9 @@ static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { { "DNSSECSupported", "b", NULL, offsetof(struct global_info, dnssec_supported) }, {} }; - _cleanup_(sd_bus_error_free) sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_(sd_bus_message_unrefp) sd_bus_message *m = NULL; + _cleanup_(global_info_clear) struct global_info global_info = {}; char **i; int r; @@ -1689,58 +1681,46 @@ static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { &error, &m, &global_info); - if (r < 0) { - log_error_errno(r, "Failed to get global data: %s", bus_error_message(&error, r)); - goto finish; - } + if (r < 0) + return log_error_errno(r, "Failed to get global data: %s", bus_error_message(&error, r)); (void) pager_open(arg_no_pager, false); - if (mode == STATUS_DNS) { - r = status_print_strv_global(global_info.dns); - goto finish; - } + if (mode == STATUS_DNS) + return status_print_strv_global(global_info.dns); - if (mode == STATUS_DOMAIN) { - r = status_print_strv_global(global_info.domains); - goto finish; - } + if (mode == STATUS_DOMAIN) + return status_print_strv_global(global_info.domains); - if (mode == STATUS_NTA) { - r = status_print_strv_global(global_info.ntas); - goto finish; - } + if (mode == STATUS_NTA) + return status_print_strv_global(global_info.ntas); if (mode == STATUS_LLMNR) { printf("%sGlobal%s: %s\n", ansi_highlight(), ansi_normal(), strna(global_info.llmnr)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_MDNS) { printf("%sGlobal%s: %s\n", ansi_highlight(), ansi_normal(), strna(global_info.mdns)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_PRIVATE) { printf("%sGlobal%s: %s\n", ansi_highlight(), ansi_normal(), strna(global_info.dns_over_tls)); - r = 0; - goto finish; + return 0; } if (mode == STATUS_DNSSEC) { printf("%sGlobal%s: %s\n", ansi_highlight(), ansi_normal(), strna(global_info.dnssec)); - r = 0; - goto finish; + return 0; } printf("%sGlobal%s\n", ansi_highlight(), ansi_normal()); @@ -1786,16 +1766,7 @@ static int status_global(sd_bus *bus, StatusMode mode, bool *empty_line) { *empty_line = true; - r = 0; - -finish: - free(global_info.current_dns); - strv_free(global_info.dns); - strv_free(global_info.fallback_dns); - strv_free(global_info.domains); - strv_free(global_info.ntas); - - return r; + return 0; } static int status_all(sd_bus *bus, StatusMode mode) { diff --git a/src/run/run.c b/src/run/run.c index 2910fcb272..9ad44e7b57 100644 --- a/src/run/run.c +++ b/src/run/run.c @@ -940,10 +940,8 @@ static int start_transient_service( &error, &pty_reply, "s", arg_host); - if (r < 0) { - log_error("Failed to get machine PTY: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to get machine PTY: %s", bus_error_message(&error, -r)); r = sd_bus_message_read(pty_reply, "hs", &master, &s); if (r < 0) @@ -1219,10 +1217,8 @@ static int start_transient_scope( polkit_agent_open_if_enabled(arg_transport, arg_ask_password); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to start transient scope unit: %s", bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to start transient scope unit: %s", bus_error_message(&error, -r)); if (arg_nice_set) { if (setpriority(PRIO_PROCESS, 0, arg_nice) < 0) @@ -1437,10 +1433,8 @@ static int start_transient_trigger( polkit_agent_open_if_enabled(arg_transport, arg_ask_password); r = sd_bus_call(bus, m, 0, &error, &reply); - if (r < 0) { - log_error("Failed to start transient %s unit: %s", suffix + 1, bus_error_message(&error, -r)); - return r; - } + if (r < 0) + return log_error_errno(r, "Failed to start transient %s unit: %s", suffix + 1, bus_error_message(&error, -r)); r = sd_bus_message_read(reply, "o", &object); if (r < 0) diff --git a/src/shared/dissect-image.c b/src/shared/dissect-image.c index fa1cf26ee1..5f19ffc4fb 100644 --- a/src/shared/dissect-image.c +++ b/src/shared/dissect-image.c @@ -1101,10 +1101,8 @@ int dissected_image_decrypt_interactively( return r; if (r == -EKEYREJECTED) log_error_errno(r, "Incorrect passphrase, try again!"); - else if (r != -ENOKEY) { - log_error_errno(r, "Failed to decrypt image: %m"); - return r; - } + else if (r != -ENOKEY) + return log_error_errno(r, "Failed to decrypt image: %m"); if (--n < 0) { log_error("Too many retries."); diff --git a/src/sulogin-shell/sulogin-shell.c b/src/sulogin-shell/sulogin-shell.c index d0e5a89f1f..5db3592d6f 100644 --- a/src/sulogin-shell/sulogin-shell.c +++ b/src/sulogin-shell/sulogin-shell.c @@ -59,9 +59,9 @@ static int start_default_target(sd_bus *bus) { "ss", "default.target", "isolate"); if (r < 0) - log_error("Failed to start default target: %s", bus_error_message(&error, r)); + return log_error_errno(r, "Failed to start default target: %s", bus_error_message(&error, r)); - return r; + return 0; } static int fork_wait(const char* const cmdline[]) { diff --git a/src/timedate/timedatectl.c b/src/timedate/timedatectl.c index befc8cb723..a541b01920 100644 --- a/src/timedate/timedatectl.c +++ b/src/timedate/timedatectl.c @@ -204,9 +204,9 @@ static int set_time(int argc, char **argv, void *userdata) { NULL, "xbb", (int64_t) t, relative, interactive); if (r < 0) - log_error("Failed to set time: %s", bus_error_message(&error, r)); + return log_error_errno(r, "Failed to set time: %s", bus_error_message(&error, r)); - return r; + return 0; } static int set_timezone(int argc, char **argv, void *userdata) { @@ -225,9 +225,9 @@ static int set_timezone(int argc, char **argv, void *userdata) { NULL, "sb", argv[1], arg_ask_password); if (r < 0) - log_error("Failed to set time zone: %s", bus_error_message(&error, r)); + return log_error_errno(r, "Failed to set time zone: %s", bus_error_message(&error, r)); - return r; + return 0; } static int set_local_rtc(int argc, char **argv, void *userdata) { @@ -250,9 +250,9 @@ static int set_local_rtc(int argc, char **argv, void *userdata) { NULL, "bbb", b, arg_adjust_system_clock, arg_ask_password); if (r < 0) - log_error("Failed to set local RTC: %s", bus_error_message(&error, r)); + return log_error_errno(r, "Failed to set local RTC: %s", bus_error_message(&error, r)); - return r; + return 0; } static int set_ntp(int argc, char **argv, void *userdata) { @@ -275,9 +275,9 @@ static int set_ntp(int argc, char **argv, void *userdata) { NULL, "bb", b, arg_ask_password); if (r < 0) - log_error("Failed to set ntp: %s", bus_error_message(&error, r)); + return log_error_errno(r, "Failed to set ntp: %s", bus_error_message(&error, r)); - return r; + return 0; } static int list_timezones(int argc, char **argv, void *userdata) { |