On Wed, Jun 20, 2018 at 3:03 AM Jonathan Marek <jonat...@marek.ca> wrote:
>
> Signed-off-by: Jonathan Marek <jonat...@marek.ca>
> ---
> src/gallium/drivers/freedreno/a2xx/fd2_program.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/gallium/drivers/freedreno/a2xx/fd2_program.c
> b/src/gallium/drivers/freedreno/a2xx/fd2_program.c
> index 9a77457251..834a7c7fcd 100644
> --- a/src/gallium/drivers/freedreno/a2xx/fd2_program.c
> +++ b/src/gallium/drivers/freedreno/a2xx/fd2_program.c
> @@ -54,6 +54,8 @@ create_shader(enum shader_t type)
> static void
> delete_shader(struct fd2_shader_stateobj *so)
> {
> + if (!so)
> + return;
> ir2_shader_destroy(so->ir);
> free(so->tokens);
> free(so->bin);
This seems to just put a band-aid on top of some other bug... Wouldn't
it be better to lift this out to the call-site instead?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
