scsi: iscsi: Fix set_param() handling
authorMike Christie <michael.christie@oracle.com>
Sun, 10 Oct 2021 16:19:04 +0000 (11:19 -0500)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 12 Oct 2021 16:51:25 +0000 (12:51 -0400)
commit187a580c9e7895978dcd1e627b9c9e7e3d13ca96
tree468ebb416224ca2934af98416a8dfe86195ee14d
parent50b6cb3516365cb69753b006be2b61c966b70588
scsi: iscsi: Fix set_param() handling

In commit 9e67600ed6b8 ("scsi: iscsi: Fix race condition between login and
sync thread") we meant to add a check where before we call ->set_param() we
make sure the iscsi_cls_connection is bound. The problem is that between
versions 4 and 5 of the patch the deletion of the unchecked set_param()
call was dropped so we ended up with 2 calls. As a result we can still hit
a crash where we access the unbound connection on the first call.

This patch removes that first call.

Fixes: 9e67600ed6b8 ("scsi: iscsi: Fix race condition between login and sync thread")
Link: https://lore.kernel.org/r/20211010161904.60471-1-michael.christie@oracle.com
Reviewed-by: Lee Duncan <lduncan@suse.com>
Reviewed-by: Li Feng <fengli@smartx.com>
Signed-off-by: Mike Christie <michael.christie@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_transport_iscsi.c