From: Nicolai Hähnle <nicolai.haehnle@amd.com>
Date: Wed, 10 May 2017 16:45:40 +0000 (+0200)
Subject: radeonsi: silence a Coverity warning
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3811730a3756fbe9d7dd2dc5974a9ef6094b86be;p=mesa.git

radeonsi: silence a Coverity warning

Coverity doesn't understand that we'll never pass non-NULL for vertex
shaders.

This is a bit lame, actually. A straightforward cross-procedural analysis
limited to this source file should be enough to prove that there's no
NULL-pointer dereference. Oh well.

CID: 1405999
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
---

diff --git a/src/gallium/drivers/radeonsi/si_debug.c b/src/gallium/drivers/radeonsi/si_debug.c
index d08a8fcf0ec..d1159ad5e27 100644
--- a/src/gallium/drivers/radeonsi/si_debug.c
+++ b/src/gallium/drivers/radeonsi/si_debug.c
@@ -478,6 +478,8 @@ static void si_dump_descriptors(struct si_context *sctx,
 	};
 
 	if (processor == PIPE_SHADER_VERTEX) {
+		assert(info); /* only CS may not have an info struct */
+
 		si_dump_descriptor_list(&sctx->vertex_buffers, shader_name[processor],
 					" - Vertex buffer", info->num_inputs, f);
 	}