diff options
| author | Dave Airlie <[email protected]> | 2017-02-10 14:03:51 +1000 |
|---|---|---|
| committer | Dave Airlie <[email protected]> | 2017-02-14 08:00:30 +1000 |
| commit | 69fc7a2c828adbb454072d386a26b568e56dd886 (patch) | |
| tree | ddd38713b0ec61da91af082246c370f51dba8386 /src/gallium/auxiliary | |
| parent | 62fef3e15984dbf278d6835895140fd788d20a38 (diff) | |
tgsi: fix memory leak in tgsi sanity check
This just fixes this without repeating the code.
Reported-by: Li Qiang
Cc: "17.0" <[email protected]>
Reviewed-by: Nicolai Hähnle <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>
Diffstat (limited to 'src/gallium/auxiliary')
| -rw-r--r-- | src/gallium/auxiliary/tgsi/tgsi_sanity.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/gallium/auxiliary/tgsi/tgsi_sanity.c b/src/gallium/auxiliary/tgsi/tgsi_sanity.c index f867925ef58..239a2c93835 100644 --- a/src/gallium/auxiliary/tgsi/tgsi_sanity.c +++ b/src/gallium/auxiliary/tgsi/tgsi_sanity.c @@ -559,6 +559,7 @@ tgsi_sanity_check( const struct tgsi_token *tokens ) { struct sanity_check_ctx ctx; + boolean retval; ctx.iter.prolog = prolog; ctx.iter.iterate_instruction = iter_instruction; @@ -580,11 +581,12 @@ tgsi_sanity_check( ctx.implied_array_size = 0; ctx.print = debug_get_option_print_sanity(); - if (!tgsi_iterate_shader( tokens, &ctx.iter )) - return FALSE; - + retval = tgsi_iterate_shader( tokens, &ctx.iter ); regs_hash_destroy(ctx.regs_decl); regs_hash_destroy(ctx.regs_used); regs_hash_destroy(ctx.regs_ind_used); + if (retval == FALSE) + return FALSE; + return ctx.errors == 0; } |