summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPeter Hutterer <peter.hutterer@who-t.net>2023-05-02 14:15:55 +1000
committerPeter Hutterer <peter.hutterer@who-t.net>2023-05-05 09:57:23 +1000
commitb06aedb8badc0ed99bad01f11015dd3ac85e50bb (patch)
treeff601686cfbace70d8074c0c3ec868e7fee05c06 /src
parent98b3314278f581f4ab9b3a5170341bdd7aea3719 (diff)
downloadxorg-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.c4
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;