diff options
| author | Dylan Baker <dylan.c.baker@intel.com> | 2025-11-06 11:49:15 -0800 |
|---|---|---|
| committer | Marge Bot <marge-bot@fdo.invalid> | 2025-11-06 21:07:11 +0000 |
| commit | 12e22d5bc1afe302206897ad7ea51dcd22f0e857 (patch) | |
| tree | 762ac521942d9e21b5f31df7281892d5ae7f3c7c /src/intel/vulkan/anv_video.c | |
| parent | d5199b07d2631581f4a1ca56dde8a8e60e5188d2 (diff) | |
anv: prevent potential, but unlikely, overflow
The code in question multiplies `uint32_t`s together and assigns them to
a `uint64_t`. It seems rather unlikely at there would be an overflow,
but we might as well do the cast.
CID: 1649587
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/38289>
Diffstat (limited to 'src/intel/vulkan/anv_video.c')
| -rw-r--r-- | src/intel/vulkan/anv_video.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel/vulkan/anv_video.c b/src/intel/vulkan/anv_video.c index 4e054ba6479..2a265f3252b 100644 --- a/src/intel/vulkan/anv_video.c +++ b/src/intel/vulkan/anv_video.c @@ -588,7 +588,7 @@ get_vp9_video_mem_size(struct anv_video_session *vid, uint32_t mem_idx) size = 32; break; case ANV_VID_MEM_VP9_SEGMENT_ID: - size = width_in_ctb * height_in_ctb; + size = (uint64_t)width_in_ctb * height_in_ctb; break; case ANV_VID_MEM_VP9_HVD_LINE_ROW_STORE: case ANV_VID_MEM_VP9_HVD_TILE_ROW_STORE: @@ -596,7 +596,7 @@ get_vp9_video_mem_size(struct anv_video_session *vid, uint32_t mem_idx) break; case ANV_VID_MEM_VP9_MV_1: case ANV_VID_MEM_VP9_MV_2: - size = (width_in_ctb * height_in_ctb * 9); + size = ((uint64_t)width_in_ctb * height_in_ctb * 9); break; default: UNREACHABLE("unknown memory"); |