]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
util: split all hostname related calls into hostname-util.c
authorLennart Poettering <lennart@poettering.net>
Mon, 18 May 2015 15:10:07 +0000 (17:10 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 18 May 2015 15:10:07 +0000 (17:10 +0200)
29 files changed:
Makefile.am
src/core/hostname-setup.c
src/firstboot/firstboot.c
src/hostname/hostnamectl.c
src/hostname/hostnamed.c
src/import/pull-dkr.c
src/journal-remote/journal-gatewayd.c
src/journal/journalctl.c
src/journal/journald-server.c
src/libsystemd-network/sd-dhcp-lease.c
src/login/pam_systemd.c
src/network/networkd-dhcp4.c
src/network/networkd-network.c
src/nspawn/nspawn.c
src/nss-myhostname/nss-myhostname.c
src/resolve/resolved-dns-scope.c
src/resolve/resolved-manager.c
src/shared/condition.c
src/shared/hostname-util.c [new file with mode: 0644]
src/shared/hostname-util.h [new file with mode: 0644]
src/shared/specifier.c
src/shared/util.c
src/shared/util.h
src/shared/utmp-wtmp.c
src/systemctl/systemctl.c
src/test/test-condition.c
src/test/test-unit-file.c
src/test/test-unit-name.c
src/test/test-util.c

index 5bcbfff5c91c09fb19d4e5ad7f77590048d347eb..5f58a1c0fae8fe274470c560a0f67d58c3aefd5d 100644 (file)
@@ -915,7 +915,9 @@ libsystemd_shared_la_SOURCES = \
        src/shared/import-util.c \
        src/shared/import-util.h \
        src/shared/sysctl-util.c \
-       src/shared/sysctl-util.h
+       src/shared/sysctl-util.h \
+       src/shared/hostname-util.h \
+       src/shared/hostname-util.c
 
 if HAVE_UTMP
 libsystemd_shared_la_SOURCES += \
index 03b0ce3b4dad805d94b2cb7de2afd3c9bb549e89..217f201d0520e481626962aea6d533301d7b0b96 100644 (file)
 #include <errno.h>
 #include <stdlib.h>
 
-#include "hostname-setup.h"
 #include "macro.h"
 #include "util.h"
 #include "log.h"
 #include "fileio.h"
+#include "hostname-util.h"
+#include "hostname-setup.h"
 
 static int read_and_strip_hostname(const char *path, char **hn) {
         char *s;
index d156d57caf0a9dffda407d94e533e81312a5f49d..cda96d484ae4311dada2ff161debb232672fc9e8 100644 (file)
@@ -36,6 +36,7 @@
 #include "locale-util.h"
 #include "ask-password-api.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 static char *arg_root = NULL;
 static char *arg_locale = NULL;  /* $LANG */
index 5218b41b41595047a879fcb7e6324e4377248dc5..69ecd61f6068f00f78bd2bfe1cca9bd10aba116e 100644 (file)
 #include <string.h>
 
 #include "sd-bus.h"
-
+#include "sd-id128.h"
+#include "hostname-util.h"
 #include "bus-util.h"
 #include "bus-error.h"
 #include "util.h"
 #include "spawn-polkit-agent.h"
 #include "build.h"
-#include "sd-id128.h"
 #include "architecture.h"
 
 static bool arg_ask_password = true;
index ddf7b8f1bd1bf7baff081044a78e4e8e5ca941d3..ab9ddc706acea45c972df20651b2fb788ef6c161 100644 (file)
@@ -33,6 +33,7 @@
 #include "bus-util.h"
 #include "event-util.h"
 #include "selinux-util.h"
+#include "hostname-util.h"
 
 #define VALID_DEPLOYMENT_CHARS (DIGITS LETTERS "-.:")
 
index 0eefec562ebbf2765aa7eeef8ee3cde2ac9a2347..f89eb88669a5378c7742997b9b860adc380f5a68 100644 (file)
@@ -38,6 +38,7 @@
 #include "import-common.h"
 #include "pull-dkr.h"
 #include "process-util.h"
+#include "hostname-util.h"
 
 typedef enum DkrProgress {
         DKR_SEARCHING,
index 07aebd8c4b4590909e441822b3d1252fc3a81327..d9450ae8cdbe7fb2714271b6d47013d62b9ed6e1 100644 (file)
@@ -42,6 +42,7 @@
 #include "build.h"
 #include "fileio.h"
 #include "sigbus.h"
+#include "hostname-util.h"
 
 static char *arg_key_pem = NULL;
 static char *arg_cert_pem = NULL;
index 627e43ba35e30c577c00dbbb4af50b4a6c91e34c..09f4e0fe38e35d9e8d600256da751f427574a9fb 100644 (file)
@@ -61,6 +61,7 @@
 #include "bus-util.h"
 #include "bus-error.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 #define DEFAULT_FSS_INTERVAL_USEC (15*USEC_PER_MINUTE)
 
index 5e07ce3c49e532ab4179b62de1c4d1a1a124a924..b3a4b530808b2a0c59745c303a3d88a1f1b90c96 100644 (file)
 #include "missing.h"
 #include "conf-parser.h"
 #include "selinux-util.h"
+#include "acl-util.h"
+#include "formats-util.h"
+#include "process-util.h"
+#include "hostname-util.h"
 #include "journal-internal.h"
 #include "journal-vacuum.h"
 #include "journal-authenticate.h"
@@ -49,9 +53,6 @@
 #include "journald-native.h"
 #include "journald-audit.h"
 #include "journald-server.h"
-#include "acl-util.h"
-#include "formats-util.h"
-#include "process-util.h"
 
 #ifdef HAVE_SELINUX
 #include <selinux/selinux.h>
index fa1fa112b12b848534e3133830d393e8727b1ed5..4b8763aab08be50fe87b31405841d96c5c3832db 100644 (file)
@@ -27,7 +27,7 @@
 #include "fileio.h"
 #include "unaligned.h"
 #include "in-addr-util.h"
-
+#include "hostname-util.h"
 #include "dhcp-protocol.h"
 #include "dhcp-lease-internal.h"
 #include "sd-dhcp-lease.h"
index 1416701ee1c3cad484f5cf241513751fd626c8d9..b5d419000c7ae3b354ec75a37abadcfd5cd5d27e 100644 (file)
@@ -42,6 +42,7 @@
 #include "bus-error.h"
 #include "formats-util.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 static int parse_argv(
                 pam_handle_t *handle,
index c99e92f56d406fbc6f5b7449de2def61ce221faa..20aee53d6e91abfc32b7adbe186d82fba2f18760 100644 (file)
@@ -22,6 +22,7 @@
 #include <netinet/ether.h>
 #include <linux/if.h>
 
+#include "hostname-util.h"
 #include "networkd-link.h"
 #include "network-internal.h"
 #include "dhcp-lease-internal.h"
index 8aa4eb2ed1734813aba919aea694a2a2828ee3a6..59470841062e5414eab613fac355bade3a782010 100644 (file)
@@ -25,6 +25,7 @@
 #include "conf-files.h"
 #include "conf-parser.h"
 #include "util.h"
+#include "hostname-util.h"
 #include "networkd.h"
 #include "networkd-netdev.h"
 #include "networkd-link.h"
index 8aa7b451bb28290f938ab5cd40130a3ca1579469..4095c77fdfd2657f5dc85c922b5b67b770fbecdb 100644 (file)
 #include <blkid/blkid.h>
 #endif
 
-#include "random-util.h"
 #include "sd-daemon.h"
 #include "sd-bus.h"
 #include "sd-id128.h"
 #include "sd-rtnl.h"
+#include "random-util.h"
 #include "log.h"
 #include "util.h"
 #include "mkdir.h"
@@ -95,6 +95,7 @@
 #include "formats-util.h"
 #include "process-util.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 #ifdef HAVE_SECCOMP
 #include "seccomp-util.h"
index fed76c62e72f1ce2e915c556e4ff60991d9782bd..31db1aaf68e328518f2fdb932ca25103b518efb6 100644 (file)
@@ -29,6 +29,7 @@
 #include "local-addresses.h"
 #include "macro.h"
 #include "nss-util.h"
+#include "hostname-util.h"
 #include "util.h"
 
 /* We use 127.0.0.2 as IPv4 address. This has the advantage over
index b2bc09266ce8842fadfe64a53736845fe97c8af5..7369cbf50f5eff638ce165581245660539543632 100644 (file)
 #include "strv.h"
 #include "socket-util.h"
 #include "af-list.h"
+#include "random-util.h"
+#include "hostname-util.h"
 #include "resolved-dns-domain.h"
 #include "resolved-dns-scope.h"
-#include "random-util.h"
 
 #define MULTICAST_RATELIMIT_INTERVAL_USEC (1*USEC_PER_SEC)
 #define MULTICAST_RATELIMIT_BURST 1000
index a10a3277bef40923de08d571f00698b00902edb5..dab737d3e48852508a05b5806735087d0e2d6cb3 100644 (file)
@@ -32,6 +32,7 @@
 #include "fileio-label.h"
 #include "ordered-set.h"
 #include "random-util.h"
+#include "hostname-util.h"
 
 #include "resolved-dns-domain.h"
 #include "resolved-conf.h"
index db12df952b143aa8bf4e36528fd9763e5e1c7c8d..9f2574c2f62d7d06728c2a45e18f72fab48a61a5 100644 (file)
@@ -35,8 +35,9 @@
 #include "ima-util.h"
 #include "selinux-util.h"
 #include "audit.h"
-#include "condition.h"
 #include "cap-list.h"
+#include "hostname-util.h"
+#include "condition.h"
 
 Condition* condition_new(ConditionType type, const char *parameter, bool trigger, bool negate) {
         Condition *c;
diff --git a/src/shared/hostname-util.c b/src/shared/hostname-util.c
new file mode 100644 (file)
index 0000000..2998fdf
--- /dev/null
@@ -0,0 +1,160 @@
+/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/
+
+/***
+  This file is part of systemd.
+
+  Copyright 2015 Lennart Poettering
+
+  systemd is free software; you can redistribute it and/or modify it
+  under the terms of the GNU Lesser General Public License as published by
+  the Free Software Foundation; either version 2.1 of the License, or
+  (at your option) any later version.
+
+  systemd is distributed in the hope that it will be useful, but
+  WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  Lesser General Public License for more details.
+
+  You should have received a copy of the GNU Lesser General Public License
+  along with systemd; If not, see <http://www.gnu.org/licenses/>.
+***/
+
+#include <sys/utsname.h>
+#include <ctype.h>
+
+#include "util.h"
+#include "hostname-util.h"
+
+bool hostname_is_set(void) {
+        struct utsname u;
+
+        assert_se(uname(&u) >= 0);
+
+        if (isempty(u.nodename))
+                return false;
+
+        /* This is the built-in kernel default host name */
+        if (streq(u.nodename, "(none)"))
+                return false;
+
+        return true;
+}
+
+char* gethostname_malloc(void) {
+        struct utsname u;
+
+        assert_se(uname(&u) >= 0);
+
+        if (isempty(u.nodename) || streq(u.nodename, "(none)"))
+                return strdup(u.sysname);
+
+        return strdup(u.nodename);
+}
+
+static bool hostname_valid_char(char c) {
+        return
+                (c >= 'a' && c <= 'z') ||
+                (c >= 'A' && c <= 'Z') ||
+                (c >= '0' && c <= '9') ||
+                c == '-' ||
+                c == '_' ||
+                c == '.';
+}
+
+bool hostname_is_valid(const char *s) {
+        const char *p;
+        bool dot;
+
+        if (isempty(s))
+                return false;
+
+        /* Doesn't accept empty hostnames, hostnames with trailing or
+         * leading dots, and hostnames with multiple dots in a
+         * sequence. Also ensures that the length stays below
+         * HOST_NAME_MAX. */
+
+        for (p = s, dot = true; *p; p++) {
+                if (*p == '.') {
+                        if (dot)
+                                return false;
+
+                        dot = true;
+                } else {
+                        if (!hostname_valid_char(*p))
+                                return false;
+
+                        dot = false;
+                }
+        }
+
+        if (dot)
+                return false;
+
+        if (p-s > HOST_NAME_MAX)
+                return false;
+
+        return true;
+}
+
+char* hostname_cleanup(char *s, bool lowercase) {
+        char *p, *d;
+        bool dot;
+
+        assert(s);
+
+        for (p = s, d = s, dot = true; *p; p++) {
+                if (*p == '.') {
+                        if (dot)
+                                continue;
+
+                        *(d++) = '.';
+                        dot = true;
+                } else if (hostname_valid_char(*p)) {
+                        *(d++) = lowercase ? tolower(*p) : *p;
+                        dot = false;
+                }
+
+        }
+
+        if (dot && d > s)
+                d[-1] = 0;
+        else
+                *d = 0;
+
+        strshorten(s, HOST_NAME_MAX);
+
+        return s;
+}
+
+bool is_localhost(const char *hostname) {
+        assert(hostname);
+
+        /* This tries to identify local host and domain names
+         * described in RFC6761 plus the redhatism of .localdomain */
+
+        return streq(hostname, "localhost") ||
+               streq(hostname, "localhost.") ||
+               streq(hostname, "localdomain.") ||
+               streq(hostname, "localdomain") ||
+               endswith(hostname, ".localhost") ||
+               endswith(hostname, ".localhost.") ||
+               endswith(hostname, ".localdomain") ||
+               endswith(hostname, ".localdomain.");
+}
+
+int sethostname_idempotent(const char *s) {
+        char buf[HOST_NAME_MAX + 1] = {};
+
+        assert(s);
+
+        if (gethostname(buf, sizeof(buf)) < 0)
+                return -errno;
+
+        if (streq(buf, s))
+                return 0;
+
+        if (sethostname(s, strlen(s)) < 0)
+                return -errno;
+
+        return 1;
+}
diff --git a/src/shared/hostname-util.h b/src/shared/hostname-util.h
new file mode 100644 (file)
index 0000000..f2821c3
--- /dev/null
@@ -0,0 +1,37 @@
+/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/
+
+#pragma once
+
+/***
+  This file is part of systemd.
+
+  Copyright 2010-2015 Lennart Poettering
+
+  systemd is free software; you can redistribute it and/or modify it
+  under the terms of the GNU Lesser General Public License as published by
+  the Free Software Foundation; either version 2.1 of the License, or
+  (at your option) any later version.
+
+  systemd is distributed in the hope that it will be useful, but
+  WITHOUT ANY WARRANTY; without even the implied warranty of
+  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  Lesser General Public License for more details.
+
+  You should have received a copy of the GNU Lesser General Public License
+  along with systemd; If not, see <http://www.gnu.org/licenses/>.
+***/
+
+#include <stdbool.h>
+
+#include "macro.h"
+
+bool hostname_is_set(void);
+
+char* gethostname_malloc(void);
+
+bool hostname_is_valid(const char *s) _pure_;
+char* hostname_cleanup(char *s, bool lowercase);
+
+bool is_localhost(const char *hostname);
+
+int sethostname_idempotent(const char *s);
index 8fbf6db5df0446166214872a8b095ca2e0dac2d2..85bd477f2dedc174dcbd7defc6f5bbfec70319a2 100644 (file)
@@ -24,6 +24,7 @@
 
 #include "macro.h"
 #include "util.h"
+#include "hostname-util.h"
 #include "specifier.h"
 
 /*
index 3060adc8a659214da2b7b5687ff4f05df541ed7a..da6343f4c44e24b33544986da72f3093b7a16eeb 100644 (file)
@@ -92,6 +92,7 @@
 #include "process-util.h"
 #include "random-util.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 /* Put this test here for a lack of better place */
 assert_cc(EAGAIN == EWOULDBLOCK);
@@ -1934,26 +1935,6 @@ int sigprocmask_many(int how, ...) {
 
         return 0;
 }
-
-char* gethostname_malloc(void) {
-        struct utsname u;
-
-        assert_se(uname(&u) >= 0);
-
-        if (!isempty(u.nodename) && !streq(u.nodename, "(none)"))
-                return strdup(u.nodename);
-
-        return strdup(u.sysname);
-}
-
-bool hostname_is_set(void) {
-        struct utsname u;
-
-        assert_se(uname(&u) >= 0);
-
-        return !isempty(u.nodename) && !streq(u.nodename, "(none)");
-}
-
 char *lookup_uid(uid_t uid) {
         long bufsize;
         char *name;
@@ -2585,79 +2566,6 @@ char* strshorten(char *s, size_t l) {
         return s;
 }
 
-static bool hostname_valid_char(char c) {
-        return
-                (c >= 'a' && c <= 'z') ||
-                (c >= 'A' && c <= 'Z') ||
-                (c >= '0' && c <= '9') ||
-                c == '-' ||
-                c == '_' ||
-                c == '.';
-}
-
-bool hostname_is_valid(const char *s) {
-        const char *p;
-        bool dot;
-
-        if (isempty(s))
-                return false;
-
-        /* Doesn't accept empty hostnames, hostnames with trailing or
-         * leading dots, and hostnames with multiple dots in a
-         * sequence. Also ensures that the length stays below
-         * HOST_NAME_MAX. */
-
-        for (p = s, dot = true; *p; p++) {
-                if (*p == '.') {
-                        if (dot)
-                                return false;
-
-                        dot = true;
-                } else {
-                        if (!hostname_valid_char(*p))
-                                return false;
-
-                        dot = false;
-                }
-        }
-
-        if (dot)
-                return false;
-
-        if (p-s > HOST_NAME_MAX)
-                return false;
-
-        return true;
-}
-
-char* hostname_cleanup(char *s, bool lowercase) {
-        char *p, *d;
-        bool dot;
-
-        for (p = s, d = s, dot = true; *p; p++) {
-                if (*p == '.') {
-                        if (dot)
-                                continue;
-
-                        *(d++) = '.';
-                        dot = true;
-                } else if (hostname_valid_char(*p)) {
-                        *(d++) = lowercase ? tolower(*p) : *p;
-                        dot = false;
-                }
-
-        }
-
-        if (dot && d > s)
-                d[-1] = 0;
-        else
-                *d = 0;
-
-        strshorten(s, HOST_NAME_MAX);
-
-        return s;
-}
-
 bool machine_name_is_valid(const char *s) {
 
         if (!hostname_is_valid(s))
@@ -5355,23 +5263,6 @@ int tempfn_random_child(const char *p, char **ret) {
         return 0;
 }
 
-/* make sure the hostname is not "localhost" */
-bool is_localhost(const char *hostname) {
-        assert(hostname);
-
-        /* This tries to identify local host and domain names
-         * described in RFC6761 plus the redhatism of .localdomain */
-
-        return streq(hostname, "localhost") ||
-               streq(hostname, "localhost.") ||
-               streq(hostname, "localdomain.") ||
-               streq(hostname, "localdomain") ||
-               endswith(hostname, ".localhost") ||
-               endswith(hostname, ".localhost.") ||
-               endswith(hostname, ".localdomain") ||
-               endswith(hostname, ".localdomain.");
-}
-
 int take_password_lock(const char *root) {
 
         struct flock flock = {
@@ -5729,26 +5620,6 @@ int free_and_strdup(char **p, const char *s) {
         return 1;
 }
 
-int sethostname_idempotent(const char *s) {
-        int r;
-        char buf[HOST_NAME_MAX + 1] = {};
-
-        assert(s);
-
-        r = gethostname(buf, sizeof(buf));
-        if (r < 0)
-                return -errno;
-
-        if (streq(buf, s))
-                return 0;
-
-        r = sethostname(s, strlen(s));
-        if (r < 0)
-                return -errno;
-
-        return 1;
-}
-
 int ptsname_malloc(int fd, char **ret) {
         size_t l = 100;
 
index 22f505c0cb3c69d139cf8ed457c2f2c4fa8d708f..a2b1ec50302c5116ab4f21d9baa2a0a702de006f 100644 (file)
@@ -351,10 +351,7 @@ char* dirname_malloc(const char *path);
 void sigset_add_many(sigset_t *ss, ...);
 int sigprocmask_many(int how, ...);
 
-bool hostname_is_set(void);
-
 char* lookup_uid(uid_t uid);
-char* gethostname_malloc(void);
 char* getlogname_malloc(void);
 char* getusername_malloc(void);
 
@@ -397,9 +394,6 @@ bool nulstr_contains(const char*nulstr, const char *needle);
 
 bool plymouth_running(void);
 
-bool hostname_is_valid(const char *s) _pure_;
-char* hostname_cleanup(char *s, bool lowercase);
-
 bool machine_name_is_valid(const char *s) _pure_;
 
 char* strshorten(char *s, size_t l);
@@ -846,8 +840,6 @@ int tempfn_xxxxxx(const char *p, char **ret);
 int tempfn_random(const char *p, char **ret);
 int tempfn_random_child(const char *p, char **ret);
 
-bool is_localhost(const char *hostname);
-
 int take_password_lock(const char *root);
 
 int is_symlink(const char *path);
@@ -864,8 +856,6 @@ int unquote_many_words(const char **p, UnquoteFlags flags, ...) _sentinel_;
 
 int free_and_strdup(char **p, const char *s);
 
-int sethostname_idempotent(const char *s);
-
 #define INOTIFY_EVENT_MAX (sizeof(struct inotify_event) + NAME_MAX + 1)
 
 #define FOREACH_INOTIFY_EVENT(e, buffer, sz) \
index aaf249dd20b634d4e5052daa87b2b51bd4290aed..8f66df771894b51de7f253722923b6ea7e79fe98 100644 (file)
@@ -29,8 +29,9 @@
 
 #include "macro.h"
 #include "path-util.h"
-#include "utmp-wtmp.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
+#include "utmp-wtmp.h"
 
 int utmp_get_runlevel(int *runlevel, int *previous) {
         struct utmpx *found, lookup = { .ut_type = RUN_LVL };
index a5b3660be5694c41fd17bc4fe8b29478ffd54c03..f8e10a4710c80b7b8eabb347ae6b53b956808bec 100644 (file)
@@ -71,6 +71,7 @@
 #include "formats-util.h"
 #include "process-util.h"
 #include "terminal-util.h"
+#include "hostname-util.h"
 
 static char **arg_types = NULL;
 static char **arg_states = NULL;
index 88147c8e0a29db3c075d58e8f6cd65644c16f0d5..b788c9532d97e4a1a84f7a9d6daf5e9adee3ea13 100644 (file)
@@ -28,6 +28,7 @@
 #include "ima-util.h"
 #include "apparmor-util.h"
 #include "smack-util.h"
+#include "hostname-util.h"
 
 static void test_condition_test_path(void) {
         Condition *condition;
index 03ca70a493bbc776c70dfbb45d0bf6f52fc7fcb5..a9711ac9f5586e8b200ff0aac73084c97602769c 100644 (file)
@@ -36,6 +36,7 @@
 #include "strv.h"
 #include "fileio.h"
 #include "test-helper.h"
+#include "hostname-util.h"
 
 static int test_unit_file_get_set(void) {
         int r;
index 5053d69d1d307038474a9bfcf4e02a042241e4c6..e5405fb7f3b7d139971b40de2cf08ef95292cd6b 100644 (file)
@@ -35,6 +35,7 @@
 #include "macro.h"
 #include "path-util.h"
 #include "test-helper.h"
+#include "hostname-util.h"
 
 static void test_unit_name_is_valid(void) {
         assert_se(unit_name_is_valid("foo.service", UNIT_NAME_ANY));
index fdb772ddda35046a654c0645d509e58e0e0c4016..9af3e757e3de520e7da50a0653fd45e079035492 100644 (file)
@@ -38,6 +38,7 @@
 #include "conf-parser.h"
 #include "virt.h"
 #include "process-util.h"
+#include "hostname-util.h"
 
 static void test_streq_ptr(void) {
         assert_se(streq_ptr(NULL, NULL));