diff options
| author | Ran Benita <ran234@gmail.com> | 2012-09-21 11:14:39 +0300 |
|---|---|---|
| committer | Daniel Stone <daniel@fooishbar.org> | 2012-09-24 09:08:53 +1000 |
| commit | c17417320939d8b252c423a203e5fe45af3b69c4 (patch) | |
| tree | 43e3622494a83e745d707653f624c0998b7c81a3 /test | |
| parent | bbaa11c6e30107932b7fdff24badfaac4d8effa0 (diff) | |
Don't choke on RMLVO layout string with holes
This old rules parser gives the same kccgst here, so in the interest of
staying compatible we shouldn't fix it there. Similarly we shouldn't
touch ParseIncludeMap, so this is the best place to handle this.
Signed-off-by: Ran Benita <ran234@gmail.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/rulescomp.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/rulescomp.c b/test/rulescomp.c index 1c16dc8..e30f539 100644 --- a/test/rulescomp.c +++ b/test/rulescomp.c @@ -110,6 +110,9 @@ int main(int argc, char *argv[]) /* 20 is not a legal group; make sure this is handled gracefully. */ assert(test_rmlvo(ctx, "evdev", "", "us:20", "", "")); + /* Don't choke on missing values in RMLVO. Should just skip them. */ + assert(test_rmlvo(ctx, "evdev", "", "us,,ca", "", "")); + assert(test_rmlvo(ctx, "", "", "", "", "")); assert(test_rmlvo(ctx, NULL, NULL, NULL, NULL, NULL)); |