diff options
| author | Everest K.C. <[email protected]> | 2024-10-23 17:33:55 -0600 |
|---|---|---|
| committer | Thomas Hellström <[email protected]> | 2024-11-14 14:55:01 +0100 |
| commit | 6d9f9115c091c88cacf78734d8ea34c8609e8680 (patch) | |
| tree | 79fcf01f7e41d6bfeb36706985fa3119af494d15 /drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.h | |
| parent | 56b70bf9ec460ad7d7d94dfb7a54a8829741e16e (diff) | |
drm/xe/guc: Fix dereference before NULL check
The pointer list->list is dereferenced before the NULL check.
Fix this by moving the NULL check outside the for loop, so that
the check is performed before the dereferencing.
The list->list pointer cannot be NULL so this has no effect on runtime.
It's just a correctness issue.
This issue was reported by Coverity Scan.
https://scan7.scan.coverity.com/#/project-view/51525/11354?selectedIssue=1600335
Fixes: 0f1fdf559225 ("drm/xe/guc: Save manual engine capture into capture list")
Signed-off-by: Everest K.C. <[email protected]>
Reviewed-by: Dan Carpenter <[email protected]>
Signed-off-by: John Harrison <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
(cherry picked from commit 2aff81e039de5b0b7ef6bdcb2c320f121f69e2b4)
Signed-off-by: Thomas Hellström <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.h')
0 files changed, 0 insertions, 0 deletions