From a06b0b562bc11e5ca2ea88074fb3b38f2503ed6b Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Sun, 9 May 2010 19:12:06 +0200 Subject: [PATCH] hostname: drop invalid chars when reading hostname from disk --- hostname-setup.c | 30 ++++++++++++++++++++++++++++-- 1 file changed, 28 insertions(+), 2 deletions(-) diff --git a/hostname-setup.c b/hostname-setup.c index b8551d980..9f6147462 100644 --- a/hostname-setup.c +++ b/hostname-setup.c @@ -44,6 +44,22 @@ #define FILENAME "/etc/conf.d/hostname" #endif +static char* strip_bad_chars(char *s) { + char *p, *d; + + for (p = s, d = s; *p; p++) + if ((*p >= 'a' && *p <= 'z') || + (*p >= 'A' && *p <= 'Z') || + (*p >= '0' && *p <= '9') || + *p == '-' || + *p == '_') + *(d++) = *p; + + *d = 0; + + return s; +} + static int read_hostname(char **hn) { #if defined(TARGET_FEDORA) || defined(TARGET_ARCH) || defined(TARGET_GENTOO) @@ -77,8 +93,11 @@ static int read_hostname(char **hn) { goto finish; } - if (!(k = delete_chars(k, "\"\'"))) { - r = -ENOMEM; + strip_bad_chars(k); + + if (k[0] == 0) { + free(k); + r = -ENOENT; goto finish; } @@ -107,6 +126,13 @@ finish: if (!k) return -ENOMEM; + strip_bad_chars(k); + + if (k[0] == 0) { + free(k); + return -NOENT; + } + *hn = k; #else -- 2.30.2