diff options
author | Mike Christie <[email protected]> | 2021-10-10 11:19:04 -0500 |
---|---|---|
committer | Martin K. Petersen <[email protected]> | 2021-10-12 12:51:25 -0400 |
commit | 187a580c9e7895978dcd1e627b9c9e7e3d13ca96 (patch) | |
tree | 468ebb416224ca2934af98416a8dfe86195ee14d /lib/test_overflow.c | |
parent | 50b6cb3516365cb69753b006be2b61c966b70588 (diff) |
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/[email protected]
Reviewed-by: Lee Duncan <[email protected]>
Reviewed-by: Li Feng <[email protected]>
Signed-off-by: Mike Christie <[email protected]>
Signed-off-by: Martin K. Petersen <[email protected]>
Diffstat (limited to 'lib/test_overflow.c')
0 files changed, 0 insertions, 0 deletions