From 90db1582a97d048769198ee1c0f367d003eeb130 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Wed, 24 Apr 2024 14:58:07 +0900 Subject: [PATCH] run/run: fix invocation ID handling Fixes a regression introduced by 8157cc0e3e33c97b406cc088cf001ca524154f64. --- src/run/run.c | 2 +- src/shared/bus-util.c | 7 +++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/run/run.c b/src/run/run.c index 9d84d063316..1a3b125c6f1 100644 --- a/src/run/run.c +++ b/src/run/run.c @@ -1610,7 +1610,7 @@ static int acquire_invocation_id(sd_bus *bus, const char *unit, sd_id128_t *ret) if (r < 0) return bus_log_parse_error(r); - return 0; + return r; /* Return true when we get a non-null invocation ID. */ } static void set_window_title(PTYForward *f) { diff --git a/src/shared/bus-util.c b/src/shared/bus-util.c index 9a480aa0e3a..dfe8b60ecdb 100644 --- a/src/shared/bus-util.c +++ b/src/shared/bus-util.c @@ -914,16 +914,15 @@ int bus_message_read_id128(sd_bus_message *m, sd_id128_t *ret) { case 0: if (ret) *ret = SD_ID128_NULL; - break; + return 0; case sizeof(sd_id128_t): if (ret) memcpy(ret, a, sz); - break; + return !memeqzero(a, sz); /* This intends to sd_id128_is_null(), but ret may be NULL, so + * let'suse memeqzero() here. */ default: return -EINVAL; } - - return 0; }