From: Jackie Liu <liuyun01@xxxxxxxxxx> By the time we get here, fcport can't be NULL, because task_work is can't be NULL, so this NULL check is not necessary, just cleanup. Reported-by: k2ci <kernel-bot@xxxxxxxxxx> Signed-off-by: Jackie Liu <liuyun01@xxxxxxxxxx> --- drivers/scsi/qla2xxx/qla_target.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c index 5258b07687a9..0ba089a753f2 100644 --- a/drivers/scsi/qla2xxx/qla_target.c +++ b/drivers/scsi/qla2xxx/qla_target.c @@ -684,7 +684,7 @@ void qla24xx_delete_sess_fn(struct work_struct *work) fc_port_t *fcport = container_of(work, struct fc_port, del_work); struct qla_hw_data *ha = NULL; - if (!fcport || !fcport->vha || !fcport->vha->hw) + if (!fcport->vha || !fcport->vha->hw) return; ha = fcport->vha->hw; -- 2.25.1
