]> Pileus Git - ~andy/linux/commit
staging: comedi: s526: fix if() check in s526_gpct_winsn()
authorH Hartley Sweeten <hartleys@visionengravers.com>
Mon, 24 Sep 2012 20:50:48 +0000 (13:50 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Sep 2012 17:21:43 +0000 (10:21 -0700)
commit67f2021fb1519057b0643d871b2afcd583bcc40d
tree9f5d4d499ed2e0985b5a95ffcec2b69a1cf4d71a
parent54a2a02ea51f606ba37f800c5e5eebb049ca6fb8
staging: comedi: s526: fix if() check in s526_gpct_winsn()

This if() check was flipped from a test for valid data params
to a test for invalid params.

As pointed out by Dan Carpenter, the orignal test was:

if ((data[1] > data[0]) && (data[0] > 0)) {

the flipped test should be:

if (data[1] <= data[0]) ...

Add the missing '='.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/s526.c