diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2025-04-12 17:39:32 +0300 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2025-04-21 10:52:53 -0400 |
| commit | 8ff7c78baeeab746432934f7de82d56706c00c50 (patch) | |
| tree | 4cf732f2763ba778072acb40099d07ca7c83aa65 /drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c | |
| parent | 987718c5598a3044ecca1b6f774900df12927ed5 (diff) | |
drm/amdgpu: Fix double free in amdgpu_userq_fence_driver_alloc()
The goto frees "fence_drv" so this is a double free bug. There is no
need to call amdgpu_seq64_free(adev, fence_drv->va) since the seq64
allocation failed so change the goto to goto free_fence_drv. Also
propagate the error code from amdgpu_seq64_alloc() instead of hard coding
it to -ENOMEM.
Fixes: e7cf21fbb277 ("drm/amdgpu: Few optimization and fixes for userq fence driver")
Reviewed-by: Arvind Yadav <arvind.yadav@amd.com>
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c')
0 files changed, 0 insertions, 0 deletions