X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=conf-parser.c;h=2cf90f2defa1ff8dec0ae4d5c7aad47d433baecd;hb=48507e6621596b0d5503fad6cd9e0685917603f5;hp=2f013e3f24c26f19a76aa7d1ec7b1d220cfbdf07;hpb=f1857be0b96016de27a725339707f8ab7326a70d;p=elogind.git
diff --git a/conf-parser.c b/conf-parser.c
index 2f013e3f2..2cf90f2de 100644
--- a/conf-parser.c
+++ b/conf-parser.c
@@ -1,5 +1,24 @@
/*-*- Mode: C; c-basic-offset: 8 -*-*/
+/***
+ This file is part of systemd.
+
+ Copyright 2010 Lennart Poettering
+
+ systemd is free software; you can redistribute it and/or modify it
+ under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 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
+ General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with systemd; If not, see .
+***/
+
#include
#include
#include
@@ -44,104 +63,57 @@ static int next_assignment(
return t->parse(filename, line, section, lvalue, rvalue, t->data, userdata);
}
- log_info("[%s:%u] Unknown lvalue '%s' in section '%s'. Ignoring.", filename, line, lvalue, strna(section));
- return 0;
-}
-
-/* Returns non-zero when c is contained in s */
-static int in_string(char c, const char *s) {
- assert(s);
-
- for (; *s; s++)
- if (*s == c)
- return 1;
+ /* Warn about unknown non-extension fields. */
+ if (!startswith(lvalue, "X-"))
+ log_info("[%s:%u] Unknown lvalue '%s' in section '%s'. Ignoring.", filename, line, lvalue, strna(section));
return 0;
}
-/* Remove all whitepsapce from the beginning and the end of *s. *s may
- * be modified. */
-static char *strip(char *s) {
- char *b = s+strspn(s, WHITESPACE);
- char *e, *l = NULL;
-
- for (e = b; *e; e++)
- if (!in_string(*e, WHITESPACE))
- l = e;
-
- if (l)
- *(l+1) = 0;
-
- return b;
-}
-
/* Parse a variable assignment line */
static int parse_line(const char *filename, unsigned line, char **section, const char* const * sections, const ConfigItem *t, char *l, void *userdata) {
- char *e, *c, *b;
+ char *e;
- b = l+strspn(l, WHITESPACE);
+ l = strstrip(l);
- if ((c = strpbrk(b, COMMENTS)))
- *c = 0;
+ if (!*l)
+ return 0;
- if (!*b)
+ if (strchr(COMMENTS, *l))
return 0;
- if (startswith(b, ".include ")) {
- char *path = NULL, *fn;
+ if (startswith(l, ".include ")) {
+ char *fn;
int r;
- fn = strip(b+9);
- if (!is_path_absolute(fn)) {
- const char *k;
-
- if ((k = strrchr(filename, '/'))) {
- char *dir;
-
- if (!(dir = strndup(filename, k-filename)))
- return -ENOMEM;
-
- if (asprintf(&path, "%s/%s", dir, fn) < 0)
- return -errno;
-
- fn = path;
- free(dir);
- }
- }
+ if (!(fn = file_in_same_dir(filename, strstrip(l+9))))
+ return -ENOMEM;
r = config_parse(fn, NULL, sections, t, userdata);
- free(path);
+ free(fn);
+
return r;
}
- if (*b == '[') {
+ if (*l == '[') {
size_t k;
char *n;
- k = strlen(b);
+ k = strlen(l);
assert(k > 0);
- if (b[k-1] != ']') {
+ if (l[k-1] != ']') {
log_error("[%s:%u] Invalid section header.", filename, line);
return -EBADMSG;
}
- if (!(n = strndup(b+1, k-2)))
+ if (!(n = strndup(l+1, k-2)))
return -ENOMEM;
- if (sections) {
- const char * const * i;
- bool good = false;
- STRV_FOREACH(i, sections)
- if (streq(*i, n)) {
- good = true;
- break;
- }
-
- if (!good) {
- free(n);
- return -EBADMSG;
- }
+ if (sections && !strv_contains((char**) sections, n)) {
+ log_error("[%s:%u] Unknown section '%s'.", filename, line, n);
+ free(n);
+ return -EBADMSG;
}
free(*section);
@@ -150,7 +122,7 @@ static int parse_line(const char *filename, unsigned line, char **section, const
return 0;
}
- if (!(e = strchr(b, '='))) {
+ if (!(e = strchr(l, '='))) {
log_error("[%s:%u] Missing '='.", filename, line);
return -EBADMSG;
}
@@ -158,7 +130,7 @@ static int parse_line(const char *filename, unsigned line, char **section, const
*e = 0;
e++;
- return next_assignment(filename, line, *section, t, strip(b), strip(e), userdata);
+ return next_assignment(filename, line, *section, t, strstrip(l), strstrip(e), userdata);
}
/* Go through the file and parse each line */
@@ -166,6 +138,7 @@ int config_parse(const char *filename, FILE *f, const char* const * sections, co
unsigned line = 0;
char *section = NULL;
int r;
+ bool ours = false;
assert(filename);
assert(t);
@@ -176,6 +149,8 @@ int config_parse(const char *filename, FILE *f, const char* const * sections, co
log_error("Failed to open configuration file '%s': %s", filename, strerror(-r));
goto finish;
}
+
+ ours = true;
}
while (!feof(f)) {
@@ -199,7 +174,7 @@ int config_parse(const char *filename, FILE *f, const char* const * sections, co
finish:
free(section);
- if (f)
+ if (f && ours)
fclose(f);
return r;
@@ -396,8 +371,12 @@ int config_parse_strv(
if (!(n = new(char*, k+1)))
return -ENOMEM;
- for (k = 0; (*sv)[k]; k++)
- n[k] = (*sv)[k];
+ if (*sv)
+ for (k = 0; (*sv)[k]; k++)
+ n[k] = (*sv)[k];
+ else
+ k = 0;
+
FOREACH_WORD_QUOTED(w, l, rvalue, state)
if (!(n[k++] = strndup(w, l)))
goto fail;