]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
scsi: scsi_debug: Drop scsi_debug_device_reset() NULL pointer checks
authorJohn Garry <john.g.garry@oracle.com>
Mon, 13 Mar 2023 09:31:07 +0000 (09:31 +0000)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 17 Mar 2023 03:25:36 +0000 (23:25 -0400)
The SCSI cmnd pointer arg would never be NULL, so drop the check. In
addition, its SCSI device pointer would never be NULL (so drop that check
also).

The only caller is scsi_try_bus_device_reset(), and the command and its
device pointer could not be NULL when calling eh_device_reset_handler()
there.

Signed-off-by: John Garry <john.g.garry@oracle.com>
Acked-by: Douglas Gilbert <dgilbert@interlog.com>
Link: https://lore.kernel.org/r/20230313093114.1498305-5-john.g.garry@oracle.com
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_debug.c

index 083d08e341622ee79335cdcde02e623f388ad92f..7b0e39d52eef17ffd0faf4fd234c4449045e75b2 100644 (file)
@@ -5372,17 +5372,16 @@ static int scsi_debug_abort(struct scsi_cmnd *SCpnt)
 
 static int scsi_debug_device_reset(struct scsi_cmnd *SCpnt)
 {
+       struct scsi_device *sdp = SCpnt->device;
+       struct sdebug_dev_info *devip = sdp->hostdata;
+
        ++num_dev_resets;
-       if (SCpnt && SCpnt->device) {
-               struct scsi_device *sdp = SCpnt->device;
-               struct sdebug_dev_info *devip =
-                               (struct sdebug_dev_info *)sdp->hostdata;
 
-               if (SDEBUG_OPT_ALL_NOISE & sdebug_opts)
-                       sdev_printk(KERN_INFO, sdp, "%s\n", __func__);
-               if (devip)
-                       set_bit(SDEBUG_UA_POR, devip->uas_bm);
-       }
+       if (SDEBUG_OPT_ALL_NOISE & sdebug_opts)
+               sdev_printk(KERN_INFO, sdp, "%s\n", __func__);
+       if (devip)
+               set_bit(SDEBUG_UA_POR, devip->uas_bm);
+
        return SUCCESS;
 }