Commit:     a96173af521a173f45d3a27fa24265081f12e978
Parent:     efc9052e01fd316b7bd6b108feff7689c927ebe9
Author:     Adrian Bunk <[EMAIL PROTECTED]>
AuthorDate: Tue Jul 31 00:28:22 2007 +0200
Committer:  Greg Kroah-Hartman <[EMAIL PROTECTED]>
CommitDate: Fri Oct 12 14:54:59 2007 -0700

    USB: Storage: usbat_check_status(): fix check-after-use
    The Coverity checker spotted that we have already oops'ed if "us"
    was NULL.
    Since "us" can't be NULL in the only caller this patch removes the
    NULL check.
    Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
    Signed-off-by: Matthew Dharm <[EMAIL PROTECTED]>
    Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>
 drivers/usb/storage/shuttle_usbat.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/drivers/usb/storage/shuttle_usbat.c 
index 5e27297..17ca4d7 100644
--- a/drivers/usb/storage/shuttle_usbat.c
+++ b/drivers/usb/storage/shuttle_usbat.c
@@ -190,9 +190,6 @@ static int usbat_check_status(struct us_data *us)
        unsigned char *reply = us->iobuf;
        int rc;
-       if (!us)
-               return USB_STOR_TRANSPORT_ERROR;
        rc = usbat_get_status(us, reply);
        if (rc != USB_STOR_XFER_GOOD)
                return USB_STOR_TRANSPORT_FAILED;
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at

Reply via email to