diff options
| author | Min Li <lm0963hack@gmail.com> | 2023-03-28 12:36:27 +0300 |
|---|---|---|
| committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2023-03-29 12:18:52 +0300 |
| commit | 49f6f6483b652108bcb73accd0204a464b922395 (patch) | |
| tree | 545ac1655b50f20d05a56f453b830a967f5cfb2a /tools/perf/scripts/python/stackcollapse.py | |
| parent | b7d70b8b06edf25c4b7526e20f5b3d11175cab81 (diff) | |
| download | linux-49f6f6483b652108bcb73accd0204a464b922395.tar.xz | |
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 <lm0963hack@gmail.com>
Fixes: f89823c21224 ("drm/i915/perf: Implement I915_PERF_ADD/REMOVE_CONFIG interface")
Cc: <stable@vger.kernel.org> # v4.14+
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230328093627.5067-1-lm0963hack@gmail.com
[tursulin: Manually added stable tag.]
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions