diff options
| author | Timothy Arceri <timothy.arceri@collabora.com> | 2016-07-28 10:24:57 +1000 |
|---|---|---|
| committer | Timothy Arceri <timothy.arceri@collabora.com> | 2016-07-28 11:05:57 +1000 |
| commit | a86aa87342221625f15fdcbec603d014cf9abba3 (patch) | |
| tree | 9c520d4fa74131e5d80c10e154bb2ed098fae6cb /src/mesa/drivers/dri/i965/brw_gs.c | |
| parent | 29d70cc964411e86bcd0bbc5591b3d82f849c9f8 (diff) | |
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 <mattst88@gmail.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_gs.c')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_gs.c | 5 |
1 files changed, 1 insertions, 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); |