[Mesa-dev] [PATCH] nouveau: Explicitly upcast operands so mulitplication is 64-bit.
Vinson Lee
vlee at freedesktop.org
Fri Sep 27 21:04:03 PDT 2013
Fixes "Unintentional integer overflow" defects reported by Coverity.
Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
src/gallium/drivers/nouveau/nouveau_vp3_video.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nouveau_vp3_video.h b/src/gallium/drivers/nouveau/nouveau_vp3_video.h
index 0193ed0..700d619 100644
--- a/src/gallium/drivers/nouveau/nouveau_vp3_video.h
+++ b/src/gallium/drivers/nouveau/nouveau_vp3_video.h
@@ -152,9 +152,9 @@ nouveau_vp3_video_addr(struct nouveau_vp3_decoder *dec, struct nouveau_vp3_video
{
uint64_t ret;
if (target)
- ret = dec->ref_stride * target->valid_ref;
+ ret = (uint64_t)dec->ref_stride * target->valid_ref;
else
- ret = dec->ref_stride * (dec->base.max_references+1);
+ ret = (uint64_t)dec->ref_stride * (dec->base.max_references+1);
return dec->ref_bo->offset + ret;
}
--
1.8.3.1
More information about the mesa-dev
mailing list