On Saturday, May 21, 2016 12:06:58 PM PDT Timothy Arceri wrote:
> We would have segfaulted in the above code if prog could be NULL.
> ---
> src/mesa/drivers/dri/i965/brw_gs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_gs.c b/src/mesa/drivers/dri/i965/
brw_gs.c
> index 4dddb86..8f5dcf3 100644
> --- a/src/mesa/drivers/dri/i965/brw_gs.c
> +++ b/src/mesa/drivers/dri/i965/brw_gs.c
> @@ -144,7 +144,7 @@ brw_codegen_gs_prog(struct brw_context *brw,
>
> brw_compute_vue_map(brw->intelScreen->devinfo,
> &prog_data.base.vue_map, outputs_written,
> - prog ? prog->SeparateShader : false);
> + prog->SeparateShader);
>
> if (unlikely(INTEL_DEBUG & DEBUG_GS))
> brw_dump_ir("geometry", prog, gs, NULL);
> Right, prog == NULL should only happen for the TCS (passthrough). Reviewed-by: Kenneth Graunke <[email protected]>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
