fdo-mirrors/xorg-libx11
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/xorg/lib/libx11.git synced 2026-05-07 14:28:05 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

xkb: Fix off-by-one error in XKeycodeToKeysym

Browse source 
The code here that made indexes greater than 3 refer to XKB symbol
groups had an off-by-one error, so it would always leave out the symbol
that should have been at index 4. Rewrite the code to fix this and
simplify the logic a bit.

Signed-off-by: Adam Sampson <ats@offog.org>
This commit is contained in:
Adam Sampson 2021-07-04 23:02:04 +01:00 • committed by Alan Coopersmith
parent ddaacd2196
commit 1c04ac5141
1 changed files with 16 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

44
src/xkb/XKBBind.c
View file

@ -113,36 +113,24 @@ XKeycodeToKeysym(Display *dpy,
return NoSymbol;
if (col > 3) {
int lastSym, tmp, nGrp;
int firstSym, nGrp, grp;
lastSym = 3;
firstSym = 4;
nGrp = XkbKeyNumGroups(xkb, kc);
if ((nGrp > 0) &&
((tmp = XkbKeyGroupWidth(xkb, kc, XkbGroup1Index)) > 2)) {
if (col <= (lastSym + tmp - 2))
return XkbKeycodeToKeysym(dpy, kc, XkbGroup1Index,
col - lastSym + 2);
lastSym += tmp - 2;
}
if ((nGrp > 1) &&
((tmp = XkbKeyGroupWidth(xkb, kc, XkbGroup2Index)) > 2)) {
if (col <= (lastSym + tmp - 2))
return XkbKeycodeToKeysym(dpy, kc, XkbGroup2Index,
col - lastSym + 2);
lastSym += tmp - 2;
}
if (nGrp > 2) {
tmp = XkbKeyGroupWidth(xkb, kc, XkbGroup3Index);
if (col <= lastSym + tmp)
return XkbKeycodeToKeysym(dpy, kc, XkbGroup3Index,
col - lastSym);
lastSym += tmp;
}
if (nGrp > 3) {
tmp = XkbKeyGroupWidth(xkb, kc, XkbGroup4Index);
if (col <= lastSym + tmp)
return XkbKeycodeToKeysym(dpy, kc, XkbGroup4Index,
col - lastSym);
for (grp = 0; grp < nGrp; grp++) {
int width = XkbKeyGroupWidth(xkb, kc, grp);
int skip = 0;
if (grp < 2) {
/* Skip the first two symbols in the first two groups, since we
* return them below for indexes 0-3. */
skip = 2;
width -= skip;
if (width < 0)
width = 0;
}
if (col < firstSym + width)
return XkbKeycodeToKeysym(dpy, kc, grp, col - firstSym + skip);
firstSym += width;
}
return NoSymbol;
}