chiark / gitweb /
conf-parser: fix strv parsing
[elogind.git] / conf-parser.c
index 78078cc0b9f47a3d9df2d54679db5f898ba4f0d2..5da55874085c2cc0604320d41eca77ca38e9135f 100644 (file)
@@ -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 <http://www.gnu.org/licenses/>.
+***/
+
 #include <string.h>
 #include <stdio.h>
 #include <errno.h>
@@ -13,6 +32,7 @@
 #include "log.h"
 
 #define COMMENTS "#;\n"
+#define NEWLINES "\n\r"
 #define LINE_MAX 4096
 
 /* Run the user supplied parser for an assignment */
@@ -44,104 +64,53 @@ static int next_assignment(
                 return t->parse(filename, line, section, lvalue, rvalue, t->data, userdata);
         }
 
-        log_error("[%s:%u] Unknown lvalue '%s' in section '%s'.", filename, line, lvalue, strna(section));
-        return -EBADMSG;
-}
-
-/* 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;
-
+        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)) {
+                        free(n);
+                        return -EBADMSG;
                 }
 
                 free(*section);
@@ -150,7 +119,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 +127,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 +135,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 +146,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 +171,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 +368,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;