Commit a96173af authored by Adrian Bunk's avatar Adrian Bunk Committed by Greg Kroah-Hartman

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: default avatarAdrian Bunk <bunk@stusta.de>
Signed-off-by: default avatarMatthew Dharm <mdharm-usb@one-eyed-alien.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent efc9052e
...@@ -190,9 +190,6 @@ static int usbat_check_status(struct us_data *us) ...@@ -190,9 +190,6 @@ static int usbat_check_status(struct us_data *us)
unsigned char *reply = us->iobuf; unsigned char *reply = us->iobuf;
int rc; int rc;
if (!us)
return USB_STOR_TRANSPORT_ERROR;
rc = usbat_get_status(us, reply); rc = usbat_get_status(us, reply);
if (rc != USB_STOR_XFER_GOOD) if (rc != USB_STOR_XFER_GOOD)
return USB_STOR_TRANSPORT_FAILED; return USB_STOR_TRANSPORT_FAILED;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment