diff options
| author | Pan Bian <[email protected]> | 2019-11-06 09:14:45 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2019-11-06 21:27:48 +0000 |
| commit | 946ab8db6953535a3a88c957db8328beacdfed9d (patch) | |
| tree | 860770973362aa44ae5e12c67c85c3f87c160e92 /drivers/gpu/drm/amd/amdgpu/amdgpu_test.c | |
| parent | drm/amdgpu: change read of GPU clock counter on Vega10 VF (diff) | |
| download | kernel-946ab8db6953535a3a88c957db8328beacdfed9d.tar.gz kernel-946ab8db6953535a3a88c957db8328beacdfed9d.zip | |
drm/amdgpu: fix potential double drop fence reference
The object fence is not set to NULL after its reference is dropped. As a
result, its reference may be dropped again if error occurs after that,
which may lead to a use after free bug. To avoid the issue, fence is
explicitly set to NULL after dropping its reference.
Acked-by: Christian König <[email protected]>
Signed-off-by: Pan Bian <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_test.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_test.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c index b66d29d5ffa2..b158230af8db 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c @@ -138,6 +138,7 @@ static void amdgpu_do_test_moves(struct amdgpu_device *adev) } dma_fence_put(fence); + fence = NULL; r = amdgpu_bo_kmap(vram_obj, &vram_map); if (r) { @@ -183,6 +184,7 @@ static void amdgpu_do_test_moves(struct amdgpu_device *adev) } dma_fence_put(fence); + fence = NULL; r = amdgpu_bo_kmap(gtt_obj[i], >t_map); if (r) { |