Mesa (9.1): radeon: Initialize variables in radeon_llvm_context_init.
Michel Dänzer
daenzer at kemper.freedesktop.org
Thu May 23 08:53:50 UTC 2013
Module: Mesa
Branch: 9.1
Commit: 3fead933019656a1252bd45c083fa30515cddada
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=3fead933019656a1252bd45c083fa30515cddada
Author: Vinson Lee <vlee at freedesktop.org>
Date: Sat May 18 00:21:52 2013 -0700
radeon: Initialize variables in radeon_llvm_context_init.
'type' was not fully initialized when calling lp_build_context_init.
Fixes "Uninitialized scalar variable" defect reported by Coverity.
NOTE: This is a candidate for the stable branches.
Signed-off-by: Vinson Lee <vlee at freedesktop.org>
Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>
(cherry picked from commit 36e2c7cc1ad5723abc65fbd7588eedeaaf94a4ba)
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
index 8902ae4..4c7e9b9 100644
--- a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
+++ b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
@@ -1126,7 +1126,9 @@ void radeon_llvm_context_init(struct radeon_llvm_context * ctx)
/* XXX: We need to revisit this.I think the correct way to do this is
* to use length = 4 here and use the elem_bld for everything. */
type.floating = TRUE;
+ type.fixed = FALSE;
type.sign = TRUE;
+ type.norm = FALSE;
type.width = 32;
type.length = 1;
More information about the mesa-commit
mailing list