diff options
| author | Nikita Zhandarovich <[email protected]> | 2025-03-11 11:14:59 +0000 |
|---|---|---|
| committer | Alex Deucher <[email protected]> | 2025-03-14 03:07:07 +0000 |
| commit | 2d52de55f9ee7aaee0e09ac443f77855989c6b68 (patch) | |
| tree | a479f57e80a8a940d5fb968e1d78f7fcd26ef289 /drivers/gpu/drm/amd/amdgpu/vi.c | |
| parent | drm/amdgpu: NULL-check BO's backing store when determining GFX12 PTE flags (diff) | |
| download | kernel-2d52de55f9ee7aaee0e09ac443f77855989c6b68.tar.gz kernel-2d52de55f9ee7aaee0e09ac443f77855989c6b68.zip | |
drm/radeon: fix uninitialized size issue in radeon_vce_cs_parse()
On the off chance that command stream passed from userspace via
ioctl() call to radeon_vce_cs_parse() is weirdly crafted and
first command to execute is to encode (case 0x03000001), the function
in question will attempt to call radeon_vce_cs_reloc() with size
argument that has not been properly initialized. Specifically, 'size'
will point to 'tmp' variable before the latter had a chance to be
assigned any value.
Play it safe and init 'tmp' with 0, thus ensuring that
radeon_vce_cs_reloc() will catch an early error in cases like these.
Found by Linux Verification Center (linuxtesting.org) with static
analysis tool SVACE.
Fixes: 2fc5703abda2 ("drm/radeon: check VCE relocation buffer range v3")
Signed-off-by: Nikita Zhandarovich <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/vi.c')
0 files changed, 0 insertions, 0 deletions