diff options
| author | Felix Kuehling <[email protected]> | 2019-10-03 21:54:57 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2019-10-07 20:09:33 +0000 |
| commit | 1995b3a35fecbf8a8078a5c3ff4b4a88eddaa5fe (patch) | |
| tree | 0bad14ae90dd05969dbbbfcb87eb0125882f5132 /drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | |
| parent | drm/amdgpu: remove redundant variable r and redundant return statement (diff) | |
| download | kernel-1995b3a35fecbf8a8078a5c3ff4b4a88eddaa5fe.tar.gz kernel-1995b3a35fecbf8a8078a5c3ff4b4a88eddaa5fe.zip | |
drm/amdgpu: Fix error handling in amdgpu_ras_recovery_init
Don't set a struct pointer to NULL before freeing its members. It's
hard to see what's happening due to a local pointer-to-pointer data
aliasing con->eh_data.
Signed-off-by: Felix Kuehling <[email protected]>
Tested-by: Philip Cox <[email protected]>
Reviewed-by: Hawking Zhang <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c index 486568ded6d6..0e2ee5869b5f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c @@ -1542,10 +1542,10 @@ int amdgpu_ras_recovery_init(struct amdgpu_device *adev) release: amdgpu_ras_release_bad_pages(adev); free: - con->eh_data = NULL; kfree((*data)->bps); kfree((*data)->bps_bo); kfree(*data); + con->eh_data = NULL; out: DRM_WARN("Failed to initialize ras recovery!\n"); |