diff options
| author | Dan Carpenter <[email protected]> | 2025-06-25 15:21:58 +0000 |
|---|---|---|
| committer | Joonas Lahtinen <[email protected]> | 2025-06-30 05:12:33 +0000 |
| commit | caa7c7a76b78ce41d347003f84975125383e6b59 (patch) | |
| tree | 75ff489cd7f537640f0d570fffd3edd80e2a07d9 /drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | |
| parent | Linux 6.16-rc4 (diff) | |
| download | kernel-caa7c7a76b78ce41d347003f84975125383e6b59.tar.gz kernel-caa7c7a76b78ce41d347003f84975125383e6b59.zip | |
drm/i915/selftests: Change mock_request() to return error pointers
There was an error pointer vs NULL bug in __igt_breadcrumbs_smoketest().
The __mock_request_alloc() function implements the
smoketest->request_alloc() function pointer. It was supposed to return
error pointers, but it propogates the NULL return from mock_request()
so in the event of a failure, it would lead to a NULL pointer
dereference.
To fix this, change the mock_request() function to return error pointers
and update all the callers to expect that.
Fixes: 52c0fdb25c7c ("drm/i915: Replace global breadcrumbs with per-context interrupt tracking")
Signed-off-by: Dan Carpenter <[email protected]>
Reviewed-by: Rodrigo Vivi <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Rodrigo Vivi <[email protected]>
(cherry picked from commit 778fa8ad5f0f23397d045c7ebca048ce8def1c43)
Signed-off-by: Joonas Lahtinen <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c')
0 files changed, 0 insertions, 0 deletions