Qiu-ji Chen 8473135f89 cdx: Fix possible UAF error in driver_override_show() ... commit 91d44c1afc upstream. Fixed a possible UAF problem in driver_override_show() in drivers/cdx/cdx.c This function driver_override_show() is part of DEVICE_ATTR_RW, which includes both driver_override_show() and driver_override_store(). These functions can be executed concurrently in sysfs. The driver_override_store() function uses driver_set_override() to update the driver_override value, and driver_set_override() internally locks the device (device_lock(dev)). If driver_override_show() reads cdx_dev->driver_override without locking, it could potentially access a freed pointer if driver_override_store() frees the string concurrently. This could lead to printing a kernel address, which is a security risk since DEVICE_ATTR can be read by all users. Additionally, a similar pattern is used in drivers/amba/bus.c, as well as many other bus drivers, where device_lock() is taken in the show function, and it has been working without issues. This potential bug was detected by our experimental static analysis tool, which analyzes locking APIs and paired functions to identify data races and atomicity violations. Fixes: 1f86a00c11 ("bus/fsl-mc: add support for 'driver_override' in the mc-bus") Cc: stable <stable@kernel.org> Signed-off-by: Qiu-ji Chen <chenqiuji666@gmail.com> Link: https://lore.kernel.org/r/20250118070833.27201-1-chenqiuji666@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2025-03-13 13:02:15 +01:00
..
controller	cdx: remove unused struct 'cdx_mcdi_copy_buffer'	2024-07-31 13:56:50 +02:00
cdx_msi.c	cdx: add MSI support for CDX bus	2024-03-07 21:52:03 +00:00
cdx.c	cdx: Fix possible UAF error in driver_override_show()	2025-03-13 13:02:15 +01:00
cdx.h	cdx: add MSI support for CDX bus	2024-03-07 21:52:03 +00:00
Kconfig
Makefile	module: Convert default symbol namespace to string literal	2025-02-08 09:57:05 +01:00