From 1056457d1132e78cbf914e8abfa661465b952412 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Sun, 4 Aug 2024 14:09:53 +0900 Subject: [PATCH] systemctl: gracefully adjust bus transport and runtime scope when --boot-loader-entry=help This fixes the following assertion: === SYSTEMD_LOG_LEVEL=debug systemctl --user -H foo --boot-loader-entry=help Assertion 'transport != BUS_TRANSPORT_REMOTE || runtime_scope == RUNTIME_SCOPE_SYSTEM' failed at src/shared/bus-util.c:284, function bus_connect_transport(). Ignoring. Failed to connect to bus: Operation not supported === Fixes a bug introduced by 97af80c5a7029c3f92e982dcf9338b9e67ad9cde. Fixes #33661. Fixes oss-fuzz#70153. --- src/systemctl/systemctl-logind.c | 8 ++++++++ test/fuzz/fuzz-systemctl-parse-argv/oss-fuzz-70153 | Bin 0 -> 67 bytes 2 files changed, 8 insertions(+) create mode 100644 test/fuzz/fuzz-systemctl-parse-argv/oss-fuzz-70153 diff --git a/src/systemctl/systemctl-logind.c b/src/systemctl/systemctl-logind.c index a727606eea7..e4ef7cf9154 100644 --- a/src/systemctl/systemctl-logind.c +++ b/src/systemctl/systemctl-logind.c @@ -432,6 +432,14 @@ int help_boot_loader_entry(void) { sd_bus *bus; int r; + /* This is called without checking runtime scope and bus transport like we do in parse_argv(). + * Loading boot entries is only supported by system scope. Let's gracefully adjust them. */ + arg_runtime_scope = RUNTIME_SCOPE_SYSTEM; + if (arg_transport == BUS_TRANSPORT_CAPSULE) { + arg_host = NULL; + arg_transport = BUS_TRANSPORT_LOCAL; + } + r = acquire_bus(BUS_FULL, &bus); if (r < 0) return r; diff --git a/test/fuzz/fuzz-systemctl-parse-argv/oss-fuzz-70153 b/test/fuzz/fuzz-systemctl-parse-argv/oss-fuzz-70153 new file mode 100644 index 0000000000000000000000000000000000000000..cb874fa6c1f0c32c821b7fb08be0a5b974f58b85 GIT binary patch literal 67 zc-mvo)h%TxD$UDDP0i8ujZR1aA`r+g&tuT_D9uZNgW6h02L@f;r2PC6-JJZyl++^K KRECVyoB{y%&lbx7 literal 0 Hc-jL100001 -- 2.47.3