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