Commit ccb6d2e5 authored Feb 10, 2011 by Tomas Winkler Committed by Greg Kroah-Hartman Feb 18, 2011

staging/easycap: don't mask return value of usb_control_msg() by 0xFF



masking return value of usb_control_msg() will mask negative
error values into positive.

Cc: Mike Thomas <rmthomas@sciolus.org>
Reported-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Reviewed-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

parent 72075789

drivers/staging/easycap/easycap_low.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -270,7 +270,7 @@ static int regget(struct usb_device *pusb_device,
		0x00,
		index, reg, reg_size, 50000);

		return 0xFF & rc;
		return rc;
		}

		static int regset(struct usb_device *pusb_device, u16 index, u16 value)