diff options
author | Sven Eden <yamakuzure@gmx.net> | 2017-05-24 13:51:31 +0200 |
---|---|---|
committer | Sven Eden <yamakuzure@gmx.net> | 2017-06-16 10:12:58 +0200 |
commit | c733f9b08d9eadc5e671185112567ada81116bdf (patch) | |
tree | 5755965b950423eae4b3c8f462acaf940a9ad684 /src/basic/hostname-util.c | |
parent | c6edc23d5f8a483e93a891bf7eb76b5f4caf2fb4 (diff) |
Prep v230: Apply missing upstream fixes and updates (2/8) src/basic.
Diffstat (limited to 'src/basic/hostname-util.c')
-rw-r--r-- | src/basic/hostname-util.c | 46 |
1 files changed, 36 insertions, 10 deletions
diff --git a/src/basic/hostname-util.c b/src/basic/hostname-util.c index 3bd3a4d7b..d8d05df3c 100644 --- a/src/basic/hostname-util.c +++ b/src/basic/hostname-util.c @@ -17,9 +17,6 @@ along with systemd; If not, see <http://www.gnu.org/licenses/>. ***/ -#if defined(__GLIBC__) -# include <bits/local_lim.h> -#endif // defined(__GLIBC__) #include <errno.h> #include <limits.h> #include <stdio.h> @@ -52,6 +49,10 @@ bool hostname_is_set(void) { char* gethostname_malloc(void) { struct utsname u; + /* This call tries to return something useful, either the actual hostname + * or it makes something up. The only reason it might fail is OOM. + * It might even return "localhost" if that's set. */ + assert_se(uname(&u) >= 0); if (isempty(u.nodename) || streq(u.nodename, "(none)")) @@ -59,6 +60,31 @@ char* gethostname_malloc(void) { return strdup(u.nodename); } + +int gethostname_strict(char **ret) { + struct utsname u; + char *k; + + /* This call will rather fail than make up a name. It will not return "localhost" either. */ + + assert_se(uname(&u) >= 0); + + if (isempty(u.nodename)) + return -ENXIO; + + if (streq(u.nodename, "(none)")) + return -ENXIO; + + if (is_localhost(u.nodename)) + return -ENXIO; + + k = strdup(u.nodename); + if (!k) + return -ENOMEM; + + *ret = k; + return 0; +} #endif // 0 static bool hostname_valid_char(char c) { @@ -127,6 +153,8 @@ char* hostname_cleanup(char *s) { assert(s); + strshorten(s, HOST_NAME_MAX); + for (p = s, d = s, dot = true; *p; p++) { if (*p == '.') { if (dot) @@ -146,8 +174,6 @@ char* hostname_cleanup(char *s) { else *d = 0; - strshorten(s, HOST_NAME_MAX); - return s; } #endif // 0 @@ -156,16 +182,16 @@ 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 */ + * described in RFC6761 plus the redhatism of localdomain */ return strcaseeq(hostname, "localhost") || strcaseeq(hostname, "localhost.") || - strcaseeq(hostname, "localdomain.") || - strcaseeq(hostname, "localdomain") || + strcaseeq(hostname, "localhost.localdomain") || + strcaseeq(hostname, "localhost.localdomain.") || endswith_no_case(hostname, ".localhost") || endswith_no_case(hostname, ".localhost.") || - endswith_no_case(hostname, ".localdomain") || - endswith_no_case(hostname, ".localdomain."); + endswith_no_case(hostname, ".localhost.localdomain") || + endswith_no_case(hostname, ".localhost.localdomain."); } #if 0 /// UNNEEDED by elogind |