diff options
| author | Jeff Garzik <jeff@garzik.org> | 2007-07-17 07:08:29 +0200 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-07-17 22:18:00 +0200 |
| commit | ae97fec3701a559929c3529e35417fab133a4d39 (patch) | |
| tree | 3eddac64147a077f5f312e24ddc801b074ac2285 /drivers/usb/misc/auerswald.c | |
| parent | [netdrvr] eepro100, ne2k-pci: abort resume if pci_enable_device() fails (diff) | |
| download | linux-ae97fec3701a559929c3529e35417fab133a4d39.tar.xz linux-ae97fec3701a559929c3529e35417fab133a4d39.zip | |
drivers/usb/misc/auerswald: fix status check, remove redundant check
1) We should only set 'actual_length' output variable if usb length is
known to be good.
2) No need to check actual_length for NULL. The only caller always
passes non-NULL value.
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/usb/misc/auerswald.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/misc/auerswald.c b/drivers/usb/misc/auerswald.c index 1fd5fc220cd7..3e22b2ff9e74 100644 --- a/drivers/usb/misc/auerswald.c +++ b/drivers/usb/misc/auerswald.c @@ -630,7 +630,7 @@ static int auerchain_start_wait_urb (pauerchain_t acp, struct urb *urb, int time } else status = urb->status; - if (actual_length) + if (status >= 0) *actual_length = urb->actual_length; return status; |