]> git.ipfire.org Git - thirdparty/linux.git/commit
scsi: target: core: Fix error path in target_setup_session()
authorBob Pearson <rpearsonhpe@gmail.com>
Tue, 13 Jun 2023 14:43:00 +0000 (09:43 -0500)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 15 Jun 2023 01:54:35 +0000 (21:54 -0400)
commit91271699228bfc66f1bc8abc0327169dc156d854
treeeaab3a79e74a77060b3ec49aa805010b23bf8561
parent31d16e712bdcaee769de4780f72ff8d6cd3f0589
scsi: target: core: Fix error path in target_setup_session()

In the error exits in target_setup_session(), if a branch is taken to
free_sess: transport_free_session() may call to target_free_cmd_counter()
and then fall through to call target_free_cmd_counter() a second time.
This can, and does, sometimes cause seg faults since the data field in
cmd_cnt->refcnt has been freed in the first call.

Fix this problem by simply returning after the call to
transport_free_session(). The second call is redundant for those cases.

Fixes: 4edba7e4a8f3 ("scsi: target: Move cmd counter allocation")
Signed-off-by: Bob Pearson <rpearsonhpe@gmail.com>
Link: https://lore.kernel.org/r/20230613144259.12890-1-rpearsonhpe@gmail.com
Reviewed-by: Mike Christie <michael.christie@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/target_core_transport.c