summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2019-12-23 16:35:44 +0100
committerLennart Poettering <lennart@poettering.net>2020-01-06 15:21:40 +0100
commit5085ef0d711f1faaacddaf5519daeb150794ea99 (patch)
tree8ebec58d0ea6464a33c90fd28fbfa2ade2b331fb
parentcore: shorten code a bit (diff)
downloadsystemd-5085ef0d711f1faaacddaf5519daeb150794ea99.tar.xz
systemd-5085ef0d711f1faaacddaf5519daeb150794ea99.zip
core: no need to eat up error
This is a method call reply. We might as well propagate the error. The worst that happens is that sd-bus logs about it.
-rw-r--r--src/core/unit.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/src/core/unit.c b/src/core/unit.c
index be92d97500..5cf16c6a79 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -3227,10 +3227,8 @@ static int get_name_owner_handler(sd_bus_message *message, void *userdata, sd_bu
}
r = sd_bus_message_read(message, "s", &new_owner);
- if (r < 0) {
- bus_log_parse_error(r);
- return 0;
- }
+ if (r < 0)
+ return bus_log_parse_error(r);
if (UNIT_VTABLE(u)->bus_name_owner_change)
UNIT_VTABLE(u)->bus_name_owner_change(u, NULL, new_owner);