chiark / gitweb /
Revert "bus-proxyd: use a loop instead of c&p"
[elogind.git] / src / bus-proxyd / bus-proxyd.c
index ce2a20bb89cfce1644d607155fd7ea9426ccef74..d1f9f761038bc37468594dba0fe5688d0ff164b1 100644 (file)
 #include "bus-message.h"
 #include "bus-util.h"
 #include "build.h"
+#include "strv.h"
+
+#define UNIX_BUS_PATH "unix:path=/run/dbus/system_bus_socket"
+#define KERNEL_BUS_PATH "kernel:path=/dev/kdbus/0-system/bus"
 
 #ifdef ENABLE_KDBUS
-const char *arg_address = "kernel:path=/dev/kdbus/0-system/bus;unix:path=/run/dbus/system_bus_socket";
+#  define DEFAULT_BUS_PATH KERNEL_BUS_PATH ";" UNIX_BUS_PATH
 #else
-const char *arg_address = "unix:path=/run/dbus/system_bus_socket";
+#  define DEFAULT_BUS_PATH UNIX_BUS_PATH
 #endif
 
+static const char *arg_address = DEFAULT_BUS_PATH;
+static char *arg_command_line_buffer = NULL;
+
 static int help(void) {
 
         printf("%s [OPTIONS...]\n\n"
                "Connect STDIO or a socket to a given bus address.\n\n"
                "  -h --help              Show this help\n"
                "     --version           Show package version\n"
-               "     --address=ADDRESS   Connect to bus specified by address\n",
+               "     --address=ADDRESS   Connect to the bus specified by ADDRESS\n"
+               "                         (default: " DEFAULT_BUS_PATH ")\n",
                program_invocation_short_name);
 
         return 0;
@@ -77,7 +85,7 @@ static int parse_argv(int argc, char *argv[]) {
         assert(argc >= 0);
         assert(argv);
 
-        while ((c = getopt_long(argc, argv, "hsup:", options, NULL)) >= 0) {
+        while ((c = getopt_long(argc, argv, "h", options, NULL)) >= 0) {
 
                 switch (c) {
 
@@ -102,14 +110,30 @@ static int parse_argv(int argc, char *argv[]) {
                 }
         }
 
+        /* If the first command line argument is only "x" characters
+         * we'll write who we are talking to into it, so that "ps" is
+         * explanatory */
+        arg_command_line_buffer = argv[optind];
+        if (argc > optind + 1 ||
+            (arg_command_line_buffer && arg_command_line_buffer[strspn(arg_command_line_buffer, "x")] != 0)) {
+                log_error("Too many arguments");
+                return -EINVAL;
+        }
+
         return 1;
 }
 
 int main(int argc, char *argv[]) {
+
+        _cleanup_bus_creds_unref_ sd_bus_creds *creds = NULL;
         _cleanup_bus_unref_ sd_bus *a = NULL, *b = NULL;
         sd_id128_t server_id;
-        bool is_unix;
         int r, in_fd, out_fd;
+        char **cmdline;
+        const char *comm;
+        bool is_unix;
+        uid_t uid;
+        pid_t pid;
 
         log_set_target(LOG_TARGET_JOURNAL_OR_KMSG);
         log_parse_environment();
@@ -201,21 +225,76 @@ int main(int argc, char *argv[]) {
                 goto finish;
         }
 
+        if (sd_bus_get_peer_creds(b, SD_BUS_CREDS_UID|SD_BUS_CREDS_PID|SD_BUS_CREDS_CMDLINE|SD_BUS_CREDS_COMM, &creds) >= 0 &&
+            sd_bus_creds_get_uid(creds, &uid) >= 0 &&
+            sd_bus_creds_get_pid(creds, &pid) >= 0 &&
+            sd_bus_creds_get_cmdline(creds, &cmdline) >= 0 &&
+            sd_bus_creds_get_comm(creds, &comm) >= 0) {
+                _cleanup_free_ char *p = NULL, *name = NULL;
+
+                name = uid_to_name(uid);
+                if (!name) {
+                        r = log_oom();
+                        goto finish;
+                }
+
+                p = strv_join(cmdline, " ");
+                if (!p) {
+                        r = log_oom();
+                        goto finish;
+                }
+
+                /* The status string gets the full command line ... */
+                sd_notifyf(false,
+                           "STATUS=Processing requests from client PID %lu (%s); UID %lu (%s)",
+                           (unsigned long) pid, p,
+                           (unsigned long) uid, name);
+
+                /* ... and the argv line only the short comm */
+                if (arg_command_line_buffer) {
+                        size_t m, w;
+
+                        m = strlen(arg_command_line_buffer);
+                        w = snprintf(arg_command_line_buffer, m,
+                                     "[PID %lu/%s; UID %lu/%s]",
+                                     (unsigned long) pid, comm,
+                                     (unsigned long) uid, name);
+
+                        if (m > w)
+                                memset(arg_command_line_buffer + w, 0, m - w);
+
+                }
+        }
+
         for (;;) {
                 _cleanup_bus_message_unref_ sd_bus_message *m = NULL;
                 int events_a, events_b, fd;
                 uint64_t timeout_a, timeout_b, t;
                 struct timespec _ts, *ts;
+                struct pollfd *pollfd;
+                int k;
 
                 r = sd_bus_process(a, &m);
                 if (r < 0) {
-                        log_error("Failed to process bus a: %s", strerror(-r));
+                        /* treat 'connection reset by peer' as clean exit condition */
+                        if (r == -ECONNRESET)
+                                r = 0;
+                        else
+                                log_error("Failed to process bus a: %s", strerror(-r));
+
                         goto finish;
                 }
 
                 if (m) {
-                        r = sd_bus_send(b, m, NULL);
-                        if (r < 0) {
+                        /* We officially got EOF, let's quit */
+                        if (sd_bus_message_is_signal(m, "org.freedesktop.DBus.Local", "Disconnected")) {
+                                r = 0;
+                                goto finish;
+                        }
+
+                        k = sd_bus_send(b, m, NULL);
+                        if (k < 0) {
+                                r = k;
                                 log_error("Failed to send message: %s", strerror(-r));
                                 goto finish;
                         }
@@ -229,13 +308,22 @@ int main(int argc, char *argv[]) {
                         /* treat 'connection reset by peer' as clean exit condition */
                         if (r == -ECONNRESET)
                                 r = 0;
+                        else
+                                log_error("Failed to process bus b: %s", strerror(-r));
 
                         goto finish;
                 }
 
                 if (m) {
-                        r = sd_bus_send(a, m, NULL);
-                        if (r < 0) {
+                        /* We officially got EOF, let's quit */
+                        if (sd_bus_message_is_signal(m, "org.freedesktop.DBus.Local", "Disconnected")) {
+                                r = 0;
+                                goto finish;
+                        }
+
+                        k = sd_bus_send(a, m, NULL);
+                        if (k < 0) {
+                                r = k;
                                 log_error("Failed to send message: %s", strerror(-r));
                                 goto finish;
                         }
@@ -292,14 +380,13 @@ int main(int argc, char *argv[]) {
                         ts = timespec_store(&_ts, t);
                 }
 
-                {
-                        struct pollfd p[3] = {
-                                {.fd = fd,            .events = events_a, },
-                                {.fd = STDIN_FILENO,  .events = events_b & POLLIN, },
-                                {.fd = STDOUT_FILENO, .events = events_b & POLLOUT, }};
+                pollfd = (struct pollfd[3]) {
+                        {.fd = fd,     .events = events_a,           },
+                        {.fd = in_fd,  .events = events_b & POLLIN,  },
+                        {.fd = out_fd, .events = events_b & POLLOUT, }
+                };
 
-                        r = ppoll(p, ELEMENTSOF(p), ts, NULL);
-                }
+                r = ppoll(pollfd, 3, ts, NULL);
                 if (r < 0) {
                         log_error("ppoll() failed: %m");
                         goto finish;
@@ -309,5 +396,8 @@ int main(int argc, char *argv[]) {
         r = 0;
 
 finish:
+        sd_bus_flush(a);
+        sd_bus_flush(b);
+
         return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
 }