From: Vinson Lee <vlee@freedesktop.org>
Date: Sat, 28 Sep 2013 06:05:54 +0000 (-0700)
Subject: r600g/sb: Move variable dereference after null check.
X-Git-Tag: upstream/10.0.5~1203
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=114ae4747549b5677cca0dbd2b8c8531d8789c49;p=platform%2Fupstream%2Fmesa.git

r600g/sb: Move variable dereference after null check.

Fixes "Deference before null check" defect reported by Coverity.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Vadim Girlin <vadimgirlin@gmail.com>
---

diff --git a/src/gallium/drivers/r600/sb/sb_ra_init.cpp b/src/gallium/drivers/r600/sb/sb_ra_init.cpp
index 0b332a9..e53aba5 100644
--- a/src/gallium/drivers/r600/sb/sb_ra_init.cpp
+++ b/src/gallium/drivers/r600/sb/sb_ra_init.cpp
@@ -395,11 +395,12 @@ void ra_init::color_bs_constraint(ra_constraint* c) {
 
 	for (vvec::iterator I = vv.begin(), E = vv.end(); I != E; ++I) {
 		value *v = *I;
-		sel_chan gpr = v->get_final_gpr();
 
 		if (!v || v->is_dead())
 			continue;
 
+		sel_chan gpr = v->get_final_gpr();
+
 		val_set interf;
 
 		if (v->chunk)