[augeas-devel] [PATCH 1/5] * src/pathx.c: disallow ', ' in names in path expressions

lutter at redhat.com lutter at redhat.com
Fri Nov 9 22:59:27 UTC 2012


From: David Lutterkort <lutter at redhat.com>

Note that this breaks backwards compatibility, since, when searching for a
node 'a,b' you now must write "match a\,b" rather than "match a,b"
---
 lenses/tests/test_device_map.aug |    2 +-
 src/pathx.c                      |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lenses/tests/test_device_map.aug b/lenses/tests/test_device_map.aug
index 2a7810a..9b5da50 100644
--- a/lenses/tests/test_device_map.aug
+++ b/lenses/tests/test_device_map.aug
@@ -21,7 +21,7 @@ module Test_device_map =
     { "128"   = "/dev/sda" }
 
   test Device_map.lns put conf after
-    set "hd2,1" "/dev/sdb1"
+    set "hd2\,1" "/dev/sdb1"
   = "# this device map was generated by anaconda
 (fd0)     /dev/fda
 (hd0)     /dev/sda
diff --git a/src/pathx.c b/src/pathx.c
index c6aa4c4..5a6bdd1 100644
--- a/src/pathx.c
+++ b/src/pathx.c
@@ -1574,7 +1574,7 @@ static void push_new_binary_op(enum binary_op op, struct state *state) {
  * Name ::= NameNoWS NameWS* NameNoWS | NameNoWS
  */
 static char *parse_name(struct state *state) {
-    static const char const follow[] = "][|/=()!";
+    static const char const follow[] = "][|/=()!,";
     const char *s = state->pos;
     char *result;
 
-- 
1.7.7.6




More information about the augeas-devel mailing list