Mesa (master): radeon: Initialize variables in radeon_llvm_context_init.

Vinson Lee vlee at kemper.freedesktop.org
Thu May 23 06:06:42 UTC 2013


Module: Mesa
Branch: master
Commit: 36e2c7cc1ad5723abc65fbd7588eedeaaf94a4ba
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=36e2c7cc1ad5723abc65fbd7588eedeaaf94a4ba

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>

---

 .../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 3f7e79f..e7a3cbf 100644
--- a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
+++ b/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
@@ -1175,7 +1175,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