diff options
| author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-05-23 13:05:59 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-30 20:41:08 +0900 |
| commit | dd64d1efdab83d0c702a27ee7893c5df492cfe1f (patch) | |
| tree | 4002b122a00f65ad9d10dc9ff18ae38e66ecc717 | |
| parent | b986be8527c7a9a84025689d5e85bc03300bb091 (diff) | |
staging: comedi: usbduxsigma: check for invalid 'ao_timer' in all cases
Make sure to check that the 'ao_timer' value is valid for both the
high speed and low speed cases in usbduxsigma_ao_cmdtest().
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/comedi/drivers/usbduxsigma.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/comedi/drivers/usbduxsigma.c b/drivers/staging/comedi/drivers/usbduxsigma.c index ed5085a70807..aa0090c55f60 100644 --- a/drivers/staging/comedi/drivers/usbduxsigma.c +++ b/drivers/staging/comedi/drivers/usbduxsigma.c @@ -994,9 +994,9 @@ static int usbduxsigma_ao_cmdtest(struct comedi_device *dev, * we get all channels at once */ devpriv->ao_timer = cmd->scan_begin_arg / 1000000; - if (devpriv->ao_timer < 1) - err |= -EINVAL; } + if (devpriv->ao_timer < 1) + err |= -EINVAL; if (cmd->stop_src == TRIG_COUNT) { /* not continuous, use counter */ |