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