diff options
| author | Rafael J. Wysocki <[email protected]> | 2024-04-26 09:10:10 +0000 |
|---|---|---|
| committer | Rafael J. Wysocki <[email protected]> | 2024-04-26 12:57:50 +0000 |
| commit | d351eb0ab04c3e8109895fc33250cebbce9c11da (patch) | |
| tree | 69d658b6e9ac585c988866c1a4c9406e13d9dbfb /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
| parent | thermal/debugfs: Fix two locking issues with thermal zone debug (diff) | |
| download | kernel-d351eb0ab04c3e8109895fc33250cebbce9c11da.tar.gz kernel-d351eb0ab04c3e8109895fc33250cebbce9c11da.zip | |
thermal/debugfs: Prevent use-after-free from occurring after cdev removal
Since thermal_debug_cdev_remove() does not run under cdev->lock, it can
run in parallel with thermal_debug_cdev_state_update() and it may free
the struct thermal_debugfs object used by the latter after it has been
checked against NULL.
If that happens, thermal_debug_cdev_state_update() will access memory
that has been freed already causing the kernel to crash.
Address this by using cdev->lock in thermal_debug_cdev_remove() around
the cdev->debugfs value check (in case the same cdev is removed at the
same time in two different threads) and its reset to NULL.
Fixes: 755113d76786 ("thermal/debugfs: Add thermal cooling device debugfs information")
Cc :6.8+ <[email protected]> # 6.8+
Signed-off-by: Rafael J. Wysocki <[email protected]>
Reviewed-by: Lukasz Luba <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
0 files changed, 0 insertions, 0 deletions