transport_{h2g,g2h,dgram,local} may become NULL on vsock_core_unregister().
Make sure a poorly timed `rmmod transport` won't lead to a NULL/stale
pointer dereference.Note that these oopses are pretty unlikely to happen in the wild. Splats were collected after sprinkling kernel with mdelay()s. Signed-off-by: Michal Luczaj <m...@rbox.co> --- Changes in v2: - Introduce a helper function to get local CIDs safely [Stefano] - Rename goto label to indicate an error path, explain why releasing vsock_register_mutex after try_module_get() is safe [Stefano] - Link to v1: https://lore.kernel.org/r/20250618-vsock-transports-toctou-v1-0-dd2d2ede9...@rbox.co --- Michal Luczaj (3): vsock: Fix transport_{g2h,h2g} TOCTOU vsock: Fix transport_* TOCTOU vsock: Fix IOCTL_VM_SOCKETS_GET_LOCAL_CID to check also `transport_local` net/vmw_vsock/af_vsock.c | 53 ++++++++++++++++++++++++++++++++++++++---------- 1 file changed, 42 insertions(+), 11 deletions(-) --- base-commit: e0fca6f2cebff539e9317a15a37dcf432e3b851a change-id: 20250523-vsock-transports-toctou-4b75d9c2a805 Best regards, -- Michal Luczaj <m...@rbox.co>
