From a86aa87342221625f15fdcbec603d014cf9abba3 Mon Sep 17 00:00:00 2001 From: Timothy Arceri Date: Thu, 28 Jul 2016 10:24:57 +1000 Subject: [PATCH] i965: remove unnecessary null check We would have hit a segfault already if this could be null. Fixes Coverity warning spotted by Matt. Reviewed-by: Matt Turner --- src/mesa/drivers/dri/i965/brw_gs.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_gs.c b/src/mesa/drivers/dri/i965/brw_gs.c index 67a2480a2a5..dd1ec4b02fe 100644 --- a/src/mesa/drivers/dri/i965/brw_gs.c +++ b/src/mesa/drivers/dri/i965/brw_gs.c @@ -163,10 +163,7 @@ brw_codegen_gs_prog(struct brw_context *brw, &prog_data, gs->Program->nir, prog, st_index, &program_size, &error_str); if (program == NULL) { - if (prog) { - ralloc_strcat(&prog->InfoLog, error_str); - } - + ralloc_strcat(&prog->InfoLog, error_str); _mesa_problem(NULL, "Failed to compile geometry shader: %s\n", error_str); ralloc_free(mem_ctx); -- 2.30.2