From 00626c3830b869259098985afa38933d77ccec72 Mon Sep 17 00:00:00 2001 From: Alan Coopersmith Date: Sat, 2 Mar 2013 09:40:22 -0800 Subject: [PATCH] unvalidated index in _XkbReadKeyActions() [CVE-2013-1997 6/15] If the X server returns key action indexes outside the range of the number of keys it told us to allocate, out of bounds memory access could occur. Reported-by: Ilja Van Sprundel Signed-off-by: Alan Coopersmith Reviewed-by: Matthieu Herrb --- src/xkb/XKBGetMap.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/xkb/XKBGetMap.c b/src/xkb/XKBGetMap.c index 4a428d3a..86ecf9db 100644 --- a/src/xkb/XKBGetMap.c +++ b/src/xkb/XKBGetMap.c @@ -269,6 +269,10 @@ Status ret = Success; symMap = &info->map->key_sym_map[rep->firstKeyAct]; for (i=0;i<(int)rep->nKeyActs;i++,symMap++) { if (numDesc[i]==0) { + if ((i + rep->firstKeyAct) > (info->max_key_code + 1)) { + ret = BadLength; + goto done; + } info->server->key_acts[i+rep->firstKeyAct]= 0; } else {