On Tue, Aug 23, 2011 at 7:00 PM, Peter Hutterer
<[email protected]> wrote:
> Signed-off-by: Peter Hutterer <[email protected]>
> ---
> static analyzer goes all "waah! waah!" about this.
>
>  src/wcmXCommand.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/wcmXCommand.c b/src/wcmXCommand.c
> index 5666bac..fe71bd1 100644
> --- a/src/wcmXCommand.c
> +++ b/src/wcmXCommand.c
> @@ -665,7 +665,7 @@ int wcmSetProperty(DeviceIntPtr dev, Atom property, 
> XIPropertyValuePtr prop,
>
>                values = (CARD32*)prop->data;
>
> -               if ((values[0] < 0) || (values[0] > 100))
> +               if (values[0] > 100)
>                        return BadValue;
>
>                if ((values[1] < 1) || (values[1] > MAX_SAMPLES))
> --
> 1.7.6
>

Acked-by: Jason Gerecke <[email protected]>

Jason

---
Day xee-nee-svsh duu-'ushtlh-ts'it;
nuu-wee-ya' duu-xan' 'vm-nvshtlh-ts'it.
Huu-chan xuu naa~-gha.

------------------------------------------------------------------------------
EMC VNX: the world's simplest storage, starting under $10K
The only unified storage solution that offers unified management 
Up to 160% more powerful than alternatives and 25% more efficient. 
Guaranteed. http://p.sf.net/sfu/emc-vnx-dev2dev
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to