diff options
| author | Min Li <[email protected]> | 2023-03-28 17:36:27 +0800 |
|---|---|---|
| committer | Jani Nikula <[email protected]> | 2023-04-03 11:36:55 +0300 |
| commit | dc30c011469165d57af9adac5baff7d767d20e5c (patch) | |
| tree | 3215c6a8e709bc8b2582d59bdcf4478e7aaca068 /net/unix/af_unix.c | |
| parent | 95d939bb97ff9be101ae4ceeb322535589da2190 (diff) | |
drm/i915: fix race condition UAF in i915_perf_add_config_ioctl
Userspace can guess the id value and try to race oa_config object creation
with config remove, resulting in a use-after-free if we dereference the
object after unlocking the metrics_lock. For that reason, unlocking the
metrics_lock must be done after we are done dereferencing the object.
Signed-off-by: Min Li <[email protected]>
Fixes: f89823c21224 ("drm/i915/perf: Implement I915_PERF_ADD/REMOVE_CONFIG interface")
Cc: <[email protected]> # v4.14+
Reviewed-by: Andi Shyti <[email protected]>
Reviewed-by: Umesh Nerlige Ramappa <[email protected]>
Signed-off-by: Tvrtko Ursulin <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
[tursulin: Manually added stable tag.]
(cherry picked from commit 49f6f6483b652108bcb73accd0204a464b922395)
Signed-off-by: Jani Nikula <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions