[PATCH 05/15] xkb: Check for unsuported comibnation of action for XkbSetMap
Peter Hutterer
peter.hutterer at who-t.net
Tue Jul 27 15:58:58 PDT 2010
On Tue, Jul 27, 2010 at 03:09:46PM +0300, Pauli Nieminen wrote:
> This prevents validation code from using unitialized values.
>
> Signed-off-by: Pauli Nieminen <ext-pauli.nieminen at nokia.com>
> ---
> xkb/xkb.c | 8 ++++++++
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/xkb/xkb.c b/xkb/xkb.c
> index 935f5ea..9a5d5b3 100644
> --- a/xkb/xkb.c
> +++ b/xkb/xkb.c
> @@ -2366,6 +2366,14 @@ _XkbSetMapChecks(ClientPtr client, DeviceIntPtr dev, xkbSetMapReq *req, char* va
> }
> }
>
> + if ((!(req->present & XkbKeySymsMask) &&
> + (req->present & XkbKeyActionsMask)) ||
> + (!(req->present & XkbKeyTypesMask) &&
> + (req->present & XkbKeySymsMask))) {
> + client->errorValue = req->present;
> + return BadMatch;
> + }
> +
> if ((req->present & XkbKeyTypesMask) &&
> (!CheckKeyTypes(client,xkb,req,(xkbKeyTypeWireDesc **)&values,
> &nTypes,mapWidths))) {
> --
> 1.6.3.3
What was the error message that triggered this patch? It's not really
obvious from the patch or from the current code.
Cheers,
Peter
More information about the xorg-devel
mailing list