diff options
author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2006-11-16 10:38:57 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-12-01 17:23:31 -0500 |
commit | 444f4f91fda54bea57a0e31098a75f54548e8b28 (patch) | |
tree | 6fbddf83ddba64b06ce7916b28c539770d3ec3c2 /drivers/media | |
parent | 1f54a6ae79ab4369f17d6bc86eaff3125395af9c (diff) |
USB: pwc-if loop fix
We should free urbs starting at [i-1] not [i].
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/video/pwc/pwc-if.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/video/pwc/pwc-if.c b/drivers/media/video/pwc/pwc-if.c index 46c114830884..83739b213e7f 100644 --- a/drivers/media/video/pwc/pwc-if.c +++ b/drivers/media/video/pwc/pwc-if.c | |||
@@ -866,11 +866,10 @@ int pwc_isoc_init(struct pwc_device *pdev) | |||
866 | } | 866 | } |
867 | if (ret) { | 867 | if (ret) { |
868 | /* De-allocate in reverse order */ | 868 | /* De-allocate in reverse order */ |
869 | while (i >= 0) { | 869 | while (i--) { |
870 | if (pdev->sbuf[i].urb != NULL) | 870 | if (pdev->sbuf[i].urb != NULL) |
871 | usb_free_urb(pdev->sbuf[i].urb); | 871 | usb_free_urb(pdev->sbuf[i].urb); |
872 | pdev->sbuf[i].urb = NULL; | 872 | pdev->sbuf[i].urb = NULL; |
873 | i--; | ||
874 | } | 873 | } |
875 | return ret; | 874 | return ret; |
876 | } | 875 | } |