[systemd-commits] src/systemctl TODO

Zbigniew Jędrzejewski-Szmek zbyszek at kemper.freedesktop.org
Wed Jan 28 18:56:26 PST 2015


 TODO                      |    3 +++
 src/systemctl/systemctl.c |    5 +++++
 2 files changed, 8 insertions(+)

New commits:
commit 3e495a6651609d0a45b62aab5c3ed5a3b40e11ab
Author: Zbigniew Jędrzejewski-Szmek <zbyszek at in.waw.pl>
Date:   Wed Jan 28 21:30:06 2015 -0500

    systemctl: refuse --host with cat
    
    This might be fixed one day, but for now it's better to fail.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1186952

diff --git a/TODO b/TODO
index 2fee133..3c5e0f7 100644
--- a/TODO
+++ b/TODO
@@ -16,6 +16,9 @@ Bugfixes:
   str_split_quoted() should return a real return code, so spawn_child can
   report the failure properly.
 
+* When systemctl --host is used, underlying ssh connection can remain open.
+  bus_close does not kill children?
+
 External:
 
 * Fedora: add an rpmlint check that verifies that all unit files in the RPM are listed in %systemd_post macros.
diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
index 66c0b99..11b0be0 100644
--- a/src/systemctl/systemctl.c
+++ b/src/systemctl/systemctl.c
@@ -4563,6 +4563,11 @@ static int cat(sd_bus *bus, char **args) {
 
         assert(args);
 
+        if (arg_host) {
+                log_error("Option --host cannot be used with 'cat'");
+                return -EINVAL;
+        }
+
         r = init_home_and_lookup_paths(&user_home, &user_runtime, &lp);
         if (r < 0)
                 return r;



More information about the systemd-commits mailing list