chiark / gitweb /
[PATCH] fix up segfaulting binaries with new klibc
[elogind.git] / udev_config.c
index 51a91d009260db4b12a4710493818059863b9e56..5e204488d2e1462cce9d63a6e54aaa59af2b0e7a 100644 (file)
@@ -34,7 +34,7 @@
 
 #include "libsysfs/sysfs/libsysfs.h"
 #include "udev.h"
-#include "udev_lib.h"
+#include "udev_utils.h"
 #include "udev_version.h"
 #include "logging.h"
 #include "namedev.h"
 /* global variables */
 char sysfs_path[SYSFS_PATH_MAX];
 char udev_root[PATH_MAX];
-char udev_db_filename[PATH_MAX+NAME_MAX];
-char udev_permissions_filename[PATH_MAX+NAME_MAX];
+char udev_db_path[PATH_MAX+NAME_MAX];
 char udev_rules_filename[PATH_MAX+NAME_MAX];
 char udev_config_filename[PATH_MAX+NAME_MAX];
-char default_mode_str[MODE_SIZE];
-char default_owner_str[OWNER_SIZE];
-char default_group_str[GROUP_SIZE];
 int udev_log;
-int udev_sleep;
 int udev_dev_d;
+int udev_hotplug_d;
 
 
-static int string_is_true(char *str)
+static int string_is_true(const char *str)
 {
        if (strcasecmp(str, "true") == 0)
                return 1;
        if (strcasecmp(str, "yes") == 0)
                return 1;
+       if (strcasecmp(str, "1") == 0)
+               return 1;
        return 0;
 }
 
 static void init_variables(void)
 {
-       /* fill up the defaults.  
-        * If any config values are specified, they will
-        * override these values. */
-       strfieldcpy(udev_root, UDEV_ROOT);
-       strfieldcpy(udev_db_filename, UDEV_DB);
-       strfieldcpy(udev_config_filename, UDEV_CONFIG_FILE);
-       strfieldcpy(udev_rules_filename, UDEV_RULES_FILE);
-       strfieldcpy(udev_permissions_filename, UDEV_PERMISSION_FILE);
-       udev_log = string_is_true(UDEV_LOG_DEFAULT);
+       const char *env;
 
-       udev_sleep = 1;
-       if (getenv("UDEV_NO_SLEEP") != NULL)
-               udev_sleep = 0;
+       /* If any config values are specified, they will override these values. */
+       strcpy(udev_root, UDEV_ROOT);
+       strcpy(udev_db_path, UDEV_DB);
+       strcpy(udev_config_filename, UDEV_CONFIG_FILE);
+       strcpy(udev_rules_filename, UDEV_RULES_FILE);
+
+       udev_log = string_is_true(UDEV_LOG_DEFAULT);
 
        udev_dev_d = 1;
-       if (getenv("UDEV_NO_DEVD") != NULL)
+       env = getenv("UDEV_NO_DEVD");
+       if (env && string_is_true(env))
                udev_dev_d = 0;
-}
-
-int parse_get_pair(char **orig_string, char **left, char **right)
-{
-       char *temp;
-       char *string = *orig_string;
-
-       if (!string)
-               return -ENODEV;
-
-       /* eat any whitespace */
-       while (isspace(*string) || *string == ',')
-               ++string;
-
-       /* split based on '=' */
-       temp = strsep(&string, "=");
-       *left = temp;
-       if (!string)
-               return -ENODEV;
 
-       /* take the right side and strip off the '"' */
-       while (isspace(*string))
-               ++string;
-       if (*string == '"')
-               ++string;
-       else
-               return -ENODEV;
-
-       temp = strsep(&string, "\"");
-       if (!string || *temp == '\0')
-               return -ENODEV;
-       *right = temp;
-       *orig_string = string;
-       
-       return 0;
+       udev_hotplug_d = 1;
+       env = getenv("UDEV_NO_HOTPLUGD");
+       if (env && string_is_true(env))
+               udev_hotplug_d = 0;
 }
 
 static int parse_config_file(void)
 {
-       char line[255];
+       char line[LINE_SIZE];
+       char *bufline;
        char *temp;
        char *variable;
        char *value;
@@ -142,32 +108,35 @@ static int parse_config_file(void)
        /* loop through the whole file */
        lineno = 0;
        cur = 0;
-       while (1) {
+       while (cur < bufsize) {
                count = buf_get_line(buf, bufsize, cur);
-
-               strncpy(line, buf + cur, count);
-               line[count] = '\0';
-               temp = line;
-               lineno++;
-
+               bufline = &buf[cur];
                cur += count+1;
-               if (cur > bufsize)
-                       break;
-
-               dbg_parse("read '%s'", temp);
+               lineno++;
 
-               /* eat the whitespace at the beginning of the line */
-               while (isspace(*temp))
-                       ++temp;
+               if (count >= LINE_SIZE) {
+                       info("line too long, conf line skipped %s, line %d",
+                                       udev_config_filename, lineno);
+                       continue;
+               }
 
-               /* empty line? */
-               if (*temp == 0x00)
+               /* eat the whitespace */
+               while ((count > 0) && isspace(bufline[0])) {
+                       bufline++;
+                       count--;
+               }
+               if (count == 0)
                        continue;
 
                /* see if this is a comment */
-               if (*temp == COMMENT_CHARACTER)
+               if (bufline[0] == COMMENT_CHARACTER)
                        continue;
 
+               strncpy(line, bufline, count);
+               line[count] = '\0';
+               temp = line;
+               dbg_parse("read '%s'", temp);
+
                retval = parse_get_pair(&temp, &variable, &value);
                if (retval != 0)
                        info("%s:%d:%Zd: error parsing '%s'",
@@ -177,39 +146,19 @@ static int parse_config_file(void)
 
                if (strcasecmp(variable, "udev_root") == 0) {
                        strfieldcpy(udev_root, value);
-                       leading_slash(udev_root);
+                       no_trailing_slash(udev_root);
                        continue;
                }
 
                if (strcasecmp(variable, "udev_db") == 0) {
-                       strfieldcpy(udev_db_filename, value);
+                       strfieldcpy(udev_db_path, value);
+                       no_trailing_slash(udev_db_path);
                        continue;
                }
 
                if (strcasecmp(variable, "udev_rules") == 0) {
                        strfieldcpy(udev_rules_filename, value);
-                       no_leading_slash(udev_rules_filename);
-                       continue;
-               }
-
-               if (strcasecmp(variable, "udev_permissions") == 0) {
-                       strfieldcpy(udev_permissions_filename, value);
-                       no_leading_slash(udev_permissions_filename);
-                       continue;
-               }
-
-               if (strcasecmp(variable, "default_mode") == 0) {
-                       strfieldcpy(default_mode_str, value);
-                       continue;
-               }
-
-               if (strcasecmp(variable, "default_owner") == 0) {
-                       strfieldcpy(default_owner_str, value);
-                       continue;
-               }
-
-               if (strcasecmp(variable, "default_group") == 0) {
-                       strfieldcpy(default_group_str, value);
+                       no_trailing_slash(udev_rules_filename);
                        continue;
                }
 
@@ -217,9 +166,6 @@ static int parse_config_file(void)
                        udev_log = string_is_true(value);
                        continue;
                }
-
-               info("%s:%d:%Zd: unknown key '%s'",
-                    udev_config_filename, lineno, temp-line, temp);
        }
 
        file_unmap(buf, bufsize);
@@ -240,7 +186,7 @@ static void get_dirs(void)
                temp = getenv("SYSFS_PATH");
                if (temp != NULL) {
                        strfieldcpy(sysfs_path, temp);
-                       no_leading_slash(sysfs_path);
+                       no_trailing_slash(sysfs_path);
                }
 
                temp = getenv("UDEV_CONFIG_FILE");
@@ -248,22 +194,13 @@ static void get_dirs(void)
                        strfieldcpy(udev_config_filename, temp);
        }
 
-       dbg("sysfs_path='%s'", sysfs_path);
-       dbg_parse("udev_root = %s", udev_root);
-       dbg_parse("udev_config_filename = %s", udev_config_filename);
-       dbg_parse("udev_db_filename = %s", udev_db_filename);
-       dbg_parse("udev_rules_filename = %s", udev_rules_filename);
-       dbg_parse("udev_permissions_filename = %s", udev_permissions_filename);
-       dbg_parse("udev_log = %d", udev_log);
-
        parse_config_file();
-
-       dbg("udev_root = %s", udev_root);
-       dbg("udev_config_filename = %s", udev_config_filename);
-       dbg("udev_db_filename = %s", udev_db_filename);
-       dbg("udev_rules_filename = %s", udev_rules_filename);
-       dbg("udev_permissions_filename = %s", udev_permissions_filename);
-       dbg("udev_log_str = %d", udev_log);
+       dbg("sysfs_path='%s'", sysfs_path);
+       dbg("udev_root='%s'", udev_root);
+       dbg("udev_config_filename='%s'", udev_config_filename);
+       dbg("udev_db_path='%s'", udev_db_path);
+       dbg("udev_rules_filename='%s'", udev_rules_filename);
+       dbg("udev_log=%d", udev_log);
 }
 
 void udev_init_config(void)