diff options
| author | Tom Rix <trix@redhat.com> | 2022-02-09 06:52:54 -0800 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2022-02-11 16:10:40 -0500 |
| commit | 574ff46f10102f65c40d24729197033a80458238 (patch) | |
| tree | e40523ed611968dc7643e97ba8c9951fa3a717ed /drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | |
| parent | 5aa71bd773c390891974b1bcbdb3b12aa17c781d (diff) | |
drm/amdkfd: fix freeing an unset pointer
clang static analysis reports this problem
kfd_chardev.c:2092:2: warning: 1st function call argument
is an uninitialized value
kvfree(bo_privs);
^~~~~~~~~~~~~~~~
When bo_buckets alloc fails, it jumps to an error handler
that frees the yet to be allocated bo_privs. Because
bo_buckets is the first error, return directly.
Fixes: 5ccbb057c0a1 ("drm/amdkfd: CRIU Implement KFD checkpoint ioctl")
Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h')
0 files changed, 0 insertions, 0 deletions