mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-05 10:31:46 +09:00
drm/amdgpu: fix dereference null return value for the function amdgpu_vm_pt_parent
[ Upstream commit 511a623fb46a6cf578c61d4f2755783c48807c77 ] The pointer parent may be NULLed by the function amdgpu_vm_pt_parent. To make the code more robust, check the pointer parent. Signed-off-by: Jesse Zhang <Jesse.Zhang@amd.com> Suggested-by: Christian König <christian.koenig@amd.com> Reviewed-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
b2ec6c4a46
commit
2f44255b89
@@ -754,11 +754,15 @@ int amdgpu_vm_pde_update(struct amdgpu_vm_update_params *params,
|
|||||||
struct amdgpu_vm_bo_base *entry)
|
struct amdgpu_vm_bo_base *entry)
|
||||||
{
|
{
|
||||||
struct amdgpu_vm_bo_base *parent = amdgpu_vm_pt_parent(entry);
|
struct amdgpu_vm_bo_base *parent = amdgpu_vm_pt_parent(entry);
|
||||||
struct amdgpu_bo *bo = parent->bo, *pbo;
|
struct amdgpu_bo *bo, *pbo;
|
||||||
struct amdgpu_vm *vm = params->vm;
|
struct amdgpu_vm *vm = params->vm;
|
||||||
uint64_t pde, pt, flags;
|
uint64_t pde, pt, flags;
|
||||||
unsigned int level;
|
unsigned int level;
|
||||||
|
|
||||||
|
if (WARN_ON(!parent))
|
||||||
|
return -EINVAL;
|
||||||
|
|
||||||
|
bo = parent->bo;
|
||||||
for (level = 0, pbo = bo->parent; pbo; ++level)
|
for (level = 0, pbo = bo->parent; pbo; ++level)
|
||||||
pbo = pbo->parent;
|
pbo = pbo->parent;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user