X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=src%2Fgetty-generator.c;h=7fac43a0baf02bcd065ad4e59457c3887e17cf54;hp=14cceb49c880198ff0b0f50bc2ac95b154fbb8a5;hb=6bc1ce40e531257ae8dd9bbbbcb48e5de0a1187b;hpb=a17b785b5fa57fba8cd658fdf710d5987413d9c2

diff --git a/src/getty-generator.c b/src/getty-generator.c
index 14cceb49c..7fac43a0b 100644
--- a/src/getty-generator.c
+++ b/src/getty-generator.c
@@ -26,13 +26,17 @@
 #include "log.h"
 #include "util.h"
 #include "unit-name.h"
+#include "virt.h"
 
-const char *arg_dest = "/tmp";
+static const char *arg_dest = "/tmp";
 
 static int add_symlink(const char *fservice, const char *tservice) {
         char *from = NULL, *to = NULL;
         int r;
 
+        assert(fservice);
+        assert(tservice);
+
         asprintf(&from, SYSTEM_DATA_UNIT_PATH "/%s", fservice);
         asprintf(&to, "%s/getty.target.wants/%s", arg_dest, tservice);
 
@@ -47,7 +51,7 @@ static int add_symlink(const char *fservice, const char *tservice) {
         r = symlink(from, to);
         if (r < 0) {
                 if (errno == EEXIST)
-                        /* In case console=hvc is passed this will very likely result in EEXIST */
+                        /* In case console=hvc0 is passed this will very likely result in EEXIST */
                         r = 0;
                 else {
                         log_error("Failed to create symlink from %s to %s: %m", from, to);
@@ -63,24 +67,51 @@ finish:
         return r;
 }
 
+static int add_serial_getty(const char *tty) {
+        char *n;
+        int r;
+
+        assert(tty);
+
+        log_debug("Automatically adding serial getty for /dev/%s.", tty);
+
+        n = unit_name_replace_instance("serial-getty@.service", tty);
+        if (!n) {
+                log_error("Out of memory");
+                return -ENOMEM;
+        }
+
+        r = add_symlink("serial-getty@.service", n);
+        free(n);
+
+        return r;
+}
+
 int main(int argc, char *argv[]) {
+
+        static const char virtualization_consoles[] =
+                "hvc0\0"
+                "xvc0\0"
+                "hvsi0\0";
+
         int r = EXIT_SUCCESS;
         char *active;
+        const char *j;
 
         if (argc > 2) {
                 log_error("This program takes one or no arguments.");
                 return EXIT_FAILURE;
         }
 
-        if (argc > 1)
-            arg_dest = argv[1];
-
-        log_set_target(LOG_TARGET_SYSLOG_OR_KMSG);
+        log_set_target(LOG_TARGET_AUTO);
         log_parse_environment();
         log_open();
 
         umask(0022);
 
+        if (argc > 1)
+            arg_dest = argv[1];
+
         if (detect_container(NULL) > 0) {
                 log_debug("Automatically adding console shell.");
 
@@ -102,40 +133,48 @@ int main(int argc, char *argv[]) {
 
                 /* Automatically add in a serial getty on the kernel
                  * console */
-                if (!tty_is_vc(tty)) {
-                        char *n;
+                if (tty_is_vc(tty))
+                        free(active);
+                else {
+                        int k;
 
                         /* We assume that gettys on virtual terminals are
                          * started via manual configuration and do this magic
                          * only for non-VC terminals. */
 
-                        log_debug("Automatically adding serial getty for /dev/%s.", tty);
+                        k = add_serial_getty(tty);
+                        free(active);
 
-                        n = unit_name_replace_instance("serial-getty@.service", tty);
-                        if (!n || add_symlink("serial-getty@.service", n) < 0)
+                        if (k < 0) {
                                 r = EXIT_FAILURE;
-
-                        free(n);
+                                goto finish;
+                        }
                 }
-
-                free(active);
         }
 
         /* Automatically add in a serial getty on the first
          * virtualizer console */
-        if (access("/sys/class/tty/hvc0", F_OK) == 0) {
-                log_debug("Automatically adding serial getty for hvc0.");
+        NULSTR_FOREACH(j, virtualization_consoles) {
+                char *p;
+                int k;
 
-                if (add_symlink("serial-getty@.service", "serial-getty@hvc0.service") < 0)
+                if (asprintf(&p, "/sys/class/tty/%s", j) < 0) {
+                        log_error("Out of memory");
                         r = EXIT_FAILURE;
+                        goto finish;
+                }
 
-        }
+                k = access(p, F_OK);
+                free(p);
 
-        if (access("/sys/class/tty/xvc0", F_OK) == 0) {
-                log_debug("Automatically adding serial getty for xvc0.");
+                if (k < 0)
+                        continue;
 
-                if (add_symlink("serial-getty@.service", "serial-getty@xvc0.service") < 0)
+                k = add_serial_getty(j);
+                if (k < 0) {
                         r = EXIT_FAILURE;
+                        goto finish;
+                }
         }
 
 finish: