diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-11-18 08:12:31 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-12-11 14:55:20 -0500 |
commit | 22a627ba81e0f75b994d37adb68761a9b9ef7186 (patch) | |
tree | bc29d68e0ce0fd7d86ecf60abfbba524d924dbc1 /drivers/usb/host | |
parent | 55f49f26821f379c451deb9fd6de8e59afb9b37e (diff) |
USB: FIX bitfield istl_flip:1, make it unsigned.
istl_flip is a signed bitfield of one bit so it can be -1 or 0.
However in drivers/usb/host/isp1362-hcd.c:1103:
finish_iso_transfers(isp1362_hcd,
&isp1362_hcd->istl_queue[isp1362_hcd->istl_flip]);
So if isp1362_hcd->istl_flip is set, the 2nd argument becomes
&isp1362_hcd->istl_queue[-1], which is invalid.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/host')
-rw-r--r-- | drivers/usb/host/isp1362.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/usb/host/isp1362.h b/drivers/usb/host/isp1362.h index 1a253ebf7e50..5151516ea1de 100644 --- a/drivers/usb/host/isp1362.h +++ b/drivers/usb/host/isp1362.h | |||
@@ -534,8 +534,8 @@ struct isp1362_hcd { | |||
534 | 534 | ||
535 | /* periodic schedule: isochronous */ | 535 | /* periodic schedule: isochronous */ |
536 | struct list_head isoc; | 536 | struct list_head isoc; |
537 | int istl_flip:1; | 537 | unsigned int istl_flip:1; |
538 | int irq_active:1; | 538 | unsigned int irq_active:1; |
539 | 539 | ||
540 | /* Schedules for the current frame */ | 540 | /* Schedules for the current frame */ |
541 | struct isp1362_ep_queue atl_queue; | 541 | struct isp1362_ep_queue atl_queue; |