[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