Commit 76e8e7d4 authored Oct 12, 2015 by Ian Abbott Committed by Greg Kroah-Hartman Oct 13, 2015

staging: comedi: make some variables unsigned in comedi_read()



In `comedi_read()`, the `n` and `m` variables are of type `int`.  Change
them to `unsigned int` as they are used to measure a positive number of
bytes.  The `count` variable is also of type `int` and holds the
returned number of bytes.  Change it to type `ssize_t` to match the
function's return type.

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent fd060c8f

drivers/staging/comedi/comedi_fops.c

+3 −1

Original line number	Diff line number	Diff line
		@@ -2450,7 +2450,9 @@ static ssize_t comedi_read(struct file file, char __user buf, size_t nbytes,
		{
		struct comedi_subdevice *s;
		struct comedi_async *async;
		int n, m, count = 0, retval = 0;
		unsigned int n, m;
		ssize_t count = 0;
		int retval = 0;
		DECLARE_WAITQUEUE(wait, current);
		struct comedi_file *cfp = file->private_data;
		struct comedi_device *dev = cfp->dev;

Admin message