udev: allow final assignments :=

[elogind.git] / udev_rules_parse.c

diff --git a/udev_rules_parse.c b/udev_rules_parse.c
index 55412dba3c5a8e7e717236ffff1f0b8e321b300f..3f07521ccb81210994cb93a0117d1cb234317cd5 100644 (file)
--- a/udev_rules_parse.c
+++ b/udev_rules_parse.c
@@ -89,6 +89,8 @@ static int get_key(char **line, char **key, enum key_operation *operation, char
                        break;
                if (linepos[0] == '!')
                        break;
+               if (linepos[0] == ':')
+                       break;
        }
 
        /* remember end of key */
@@ -115,6 +117,10 @@ static int get_key(char **line, char **key, enum key_operation *operation, char
                *operation = KEY_OP_ASSIGN;
                linepos++;
                dbg("operator=assign");
+       } else if (linepos[0] == ':' && linepos[1] == '=') {
+               *operation = KEY_OP_ASSIGN_FINAL;
+               linepos += 2;
+               dbg("operator=assign_final");
        } else
                return -1;
 
@@ -157,7 +163,7 @@ static char *get_key_attribute(char *str)
                attr++;
                pos = strchr(attr, '}');
                if (pos == NULL) {
-                       dbg("missing closing brace for format");
+                       err("missing closing brace for format");
                        return NULL;
                }
                pos[0] = '\0';
@@ -185,7 +191,7 @@ static int rules_parse(const char *filename)
        struct udev_rule rule;
 
        if (file_map(filename, &buf, &bufsize) != 0) {
-               dbg("can't open '%s' as rules file", filename);
+               err("can't open '%s' as rules file", filename);
                return -1;
        }
        dbg("reading '%s' as rules file", filename);
@@ -236,7 +242,7 @@ static int rules_parse(const char *filename)
                while (1) {
                        char *key;
                        char *value;
-                       enum key_operation operation = KEY_OP_UNKNOWN;
+                       enum key_operation operation = KEY_OP_UNSET;
 
                        retval = get_key(&linepos, &key, &operation, &value);
                        if (retval)
@@ -256,6 +262,13 @@ static int rules_parse(const char *filename)
                                continue;
                        }
 
+                       if (strcasecmp(key, KEY_ACTION) == 0) {
+                               strlcpy(rule.action, value, sizeof(rule.action));
+                               rule.action_operation = operation;
+                               valid = 1;
+                               continue;
+                       }
+
                        if (strcasecmp(key, KEY_BUS) == 0) {
                                strlcpy(rule.bus, value, sizeof(rule.bus));
                                rule.bus_operation = operation;
@@ -274,13 +287,13 @@ static int rules_parse(const char *filename)
                                struct key_pair *pair;
 
                                if (rule.sysfs_pair_count >= KEY_SYSFS_PAIRS_MAX) {
-                                       dbg("skip rule, to many " KEY_SYSFS " keys in a single rule");
+                                       err("skip rule, to many " KEY_SYSFS " keys in a single rule");
                                        goto error;
                                }
                                pair = &rule.sysfs_pair[rule.sysfs_pair_count];
                                attr = get_key_attribute(key + sizeof(KEY_SYSFS)-1);
                                if (attr == NULL) {
-                                       dbg("error parsing " KEY_SYSFS " attribute");
+                                       err("error parsing " KEY_SYSFS " attribute");
                                        goto error;
                                }
                                strlcpy(pair->name, attr, sizeof(pair->name));
@@ -295,13 +308,13 @@ static int rules_parse(const char *filename)
                                struct key_pair *pair;
 
                                if (rule.env_pair_count >= KEY_ENV_PAIRS_MAX) {
-                                       dbg("skip rule, to many " KEY_ENV " keys in a single rule");
+                                       err("skip rule, to many " KEY_ENV " keys in a single rule");
                                        goto error;
                                }
                                pair = &rule.env_pair[rule.env_pair_count];
                                attr = get_key_attribute(key + sizeof(KEY_ENV)-1);
                                if (attr == NULL) {
-                                       dbg("error parsing " KEY_ENV " attribute");
+                                       err("error parsing " KEY_ENV " attribute");
                                        continue;
                                }
                                strlcpy(pair->name, attr, sizeof(pair->name));
@@ -357,24 +370,35 @@ static int rules_parse(const char *filename)
 
                        if (strcasecmp(key, KEY_SYMLINK) == 0) {
                                strlcpy(rule.symlink, value, sizeof(rule.symlink));
+                               rule.symlink_operation = operation;
                                valid = 1;
                                continue;
                        }
 
                        if (strcasecmp(key, KEY_OWNER) == 0) {
                                strlcpy(rule.owner, value, sizeof(rule.owner));
+                               rule.owner_operation = operation;
                                valid = 1;
                                continue;
                        }
 
                        if (strcasecmp(key, KEY_GROUP) == 0) {
                                strlcpy(rule.group, value, sizeof(rule.group));
+                               rule.group_operation = operation;
                                valid = 1;
                                continue;
                        }
 
                        if (strcasecmp(key, KEY_MODE) == 0) {
                                rule.mode = strtol(value, NULL, 8);
+                               rule.mode_operation = operation;
+                               valid = 1;
+                               continue;
+                       }
+
+                       if (strcasecmp(key, KEY_RUN) == 0) {
+                               strlcpy(rule.run, value, sizeof(rule.run));
+                               rule.run_operation = operation;
                                valid = 1;
                                continue;
                        }
@@ -400,7 +424,7 @@ static int rules_parse(const char *filename)
                                continue;
                        }
 
-                       dbg("unknown key '%s'", key);
+                       err("unknown key '%s'", key);
                        goto error;
                }
 
@@ -408,16 +432,8 @@ static int rules_parse(const char *filename)
                if (!valid)
                        goto error;
 
-               /* simple plausibility checks for given keys */
-               if ((rule.sysfs_pair[0].name[0] == '\0') ^
-                   (rule.sysfs_pair[0].value[0] == '\0')) {
-                       info("inconsistency in " KEY_SYSFS " key");
-                       goto error;
-               }
-
                if ((rule.result[0] != '\0') && (program_given == 0)) {
-                       info(KEY_RESULT " is only useful when "
-                            KEY_PROGRAM " is called in any rule before");
+                       info(KEY_RESULT " is only useful when " KEY_PROGRAM " is called in any rule before");
                        goto error;
                }
 
@@ -428,7 +444,7 @@ static int rules_parse(const char *filename)
                        dbg("add_config_dev returned with error %d", retval);
                        continue;
 error:
-                       info("parse error %s, line %d:%d, rule skipped",
+                       err("parse error %s, line %d:%d, rule skipped",
                             filename, lineno, (int) (linepos - line));
                }
        }