diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2023-05-02 14:15:55 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2023-05-05 09:57:23 +1000 |
commit | b06aedb8badc0ed99bad01f11015dd3ac85e50bb (patch) | |
tree | ff601686cfbace70d8074c0c3ec868e7fee05c06 /src | |
parent | 98b3314278f581f4ab9b3a5170341bdd7aea3719 (diff) | |
download | xorg-lib-libxkbcommon-b06aedb8badc0ed99bad01f11015dd3ac85e50bb.tar.gz |
scanner: allow for a zero terminated string as keymap
As the documentation for xkb_keymap_new_from_buffer() states, the "input string
does not have to be zero-terminated". The actual implementation however failed
with "unrecognized token/syntax error" when it encountered a null byte.
Fix this by allowing a null byte at the last position of the buffer. Anything
else is likely a client error anyway.
Fixes #307
Diffstat (limited to 'src')
-rw-r--r-- | src/keymap.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/keymap.c b/src/keymap.c index d2baf94..0291aed 100644 --- a/src/keymap.c +++ b/src/keymap.c @@ -194,6 +194,10 @@ xkb_keymap_new_from_buffer(struct xkb_context *ctx, if (!keymap) return NULL; + /* Allow a zero-terminated string as a buffer */ + if (length > 0 && buffer[length - 1] == '\0') + length--; + if (!ops->keymap_new_from_string(keymap, buffer, length)) { xkb_keymap_unref(keymap); return NULL; |