Re: [PATCH 1/1] RDMA/srpt: Do not register event handler until srpt device is fully setup

From: Bart Van Assche
Date: Thu Feb 01 2024 - 09:48:57 EST


On 1/30/24 22:24, William Kucharski wrote:
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.

A Fixes: tag is missing. Otherwise this patch looks good to me. Hence:

Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>