scsi: qedf: Fix null ptr reference in qedf_stag_change_work

Ye Bin · martinkpetersen · commit f308a35f547c · 2020-08-24T22:58:38.000-04:00
Link: https://lore.kernel.org/r/20200824033436.45570-1-yebin10@huawei.com Acked-by: Saurav Kashyap <skashyap@marvell.com> Signed-off-by: Ye Bin <yebin10@huawei.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
diff --git a/drivers/scsi/qedf/qedf_main.c b/drivers/scsi/qedf/qedf_main.c
@@ -3863,7 +3863,7 @@ void qedf_stag_change_work(struct work_struct *work)
 	    container_of(work, struct qedf_ctx, stag_work.work);
 
 	if (!qedf) {
-		QEDF_ERR(&qedf->dbg_ctx, "qedf is NULL");
+		QEDF_ERR(NULL, "qedf is NULL");
 		return;
 	}
 	QEDF_ERR(&qedf->dbg_ctx, "Performing software context reset.\n");

Original file line number	Diff line number	Diff line change
`@@ -3863,7 +3863,7 @@ void qedf_stag_change_work(struct work_struct *work)`
`3863`	`3863`	`container_of(work, struct qedf_ctx, stag_work.work);`
`3864`	`3864`
`3865`	`3865`	`if (!qedf) {`
`3866`		`- QEDF_ERR(&qedf->dbg_ctx, "qedf is NULL");`
	`3866`	`+ QEDF_ERR(NULL, "qedf is NULL");`
`3867`	`3867`	`return;`
`3868`	`3868`	`}`
`3869`	`3869`	`QEDF_ERR(&qedf->dbg_ctx, "Performing software context reset.\n");`