summaryrefslogtreecommitdiff
path: root/include/linux/device
diff options
context:
space:
mode:
authorDanilo Krummrich <dakr@kernel.org>2026-04-18 19:22:18 +0300
committerDanilo Krummrich <dakr@kernel.org>2026-04-29 01:35:38 +0300
commite9506871a8ea304cde48ff4a57226df2aadddae3 (patch)
tree8e7a76755611a5acadc413647d8dbd1e0a3a75d8 /include/linux/device
parentc9447f101315bcec9b6603de0be59ca0e6b8b505 (diff)
downloadlinux-e9506871a8ea304cde48ff4a57226df2aadddae3.tar.xz
driver core: use READ_ONCE() for dev->driver in dev_has_sync_state()
dev_has_sync_state() reads dev->driver twice without holding device_lock() -- once for the NULL check and once to dereference ->sync_state. Some callers only hold device_links_write_lock, which doesn't prevent a concurrent unbind from clearing dev->driver via device_unbind_cleanup(). Fix it by reading dev->driver exactly once with READ_ONCE(), pairing with the WRITE_ONCE() in device_set_driver(). Link: https://lore.kernel.org/driver-core/DHW8QPU1VU1F.3P6PH69HLFBYC@kernel.org/ Fixes: ac338acf514e ("driver core: Add dev_has_sync_state()") Reviewed-by: Rafael J. Wysocki (Intel) <rafael@kernel.org> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Reviewed-by: Saravana Kannan <saravanak@kernel.org> Link: https://patch.msgid.link/20260418162221.1121873-1-dakr@kernel.org Signed-off-by: Danilo Krummrich <dakr@kernel.org>
Diffstat (limited to 'include/linux/device')
0 files changed, 0 insertions, 0 deletions