Commit 306c2cde authored by William Kucharski's avatar William Kucharski Committed by Zhengchao Shao
Browse files

RDMA/srpt: Do not register event handler until srpt device is fully setup 

stable inclusion
from stable-v5.10.214
commit bdd895e0190c464f54f84579e7535d80276f0fc5
category: bugfix
bugzilla: https://gitee.com/src-openeuler/kernel/issues/I9HK8Y
CVE: CVE-2024-26872

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=bdd895e0190c464f54f84579e7535d80276f0fc5



--------------------------------

[ Upstream commit c21a8870c98611e8f892511825c9607f1e2cd456 ]

Upon rare occasions, KASAN reports a use-after-free Write
in srpt_refresh_port().

This seems to be because an event handler is registered before the
srpt device is fully setup and a race condition upon error may leave a
partially setup event handler in place.

Instead, only register the event handler after srpt device initialization
is complete.

Fixes: a42d985b ("ib_srpt: Initial SRP Target merge for v3.3-rc1")
Signed-off-by: default avatarWilliam Kucharski <william.kucharski@oracle.com>
Link: https://lore.kernel.org/r/20240202091549.991784-2-william.kucharski@oracle.com


Reviewed-by: default avatarBart Van Assche <bvanassche@acm.org>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarZhengchao Shao <shaozhengchao@huawei.com>
parent b05f2de3

drivers/infiniband/ulp/srpt/ib_srpt.c

+1 −2
Original line number Diff line number Diff line
@@ -3204,7 +3204,6 @@ static int srpt_add_one(struct ib_device *device) 


	INIT_IB_EVENT_HANDLER(&sdev->event_handler, sdev->device,

			      srpt_event_handler);

	ib_register_event_handler(&sdev->event_handler);



	for (i = 1; i <= sdev->device->phys_port_cnt; i++) {

		sport = &sdev->port[i - 1];
@@ -3227,6 +3226,7 @@ static int srpt_add_one(struct ib_device *device) 
		}

	}



	ib_register_event_handler(&sdev->event_handler);

	spin_lock(&srpt_dev_lock);

	list_add_tail(&sdev->list, &srpt_dev_list);

	spin_unlock(&srpt_dev_lock);
@@ -3237,7 +3237,6 @@ static int srpt_add_one(struct ib_device *device) 


err_port:

	srpt_unregister_mad_agent(sdev, i);

	ib_unregister_event_handler(&sdev->event_handler);

err_cm:

	if (sdev->cm_id)

		ib_destroy_cm_id(sdev->cm_id);