Re: [PATCH 4/5] usp10: Avoid leaking "open_stack" and "stack_index" in case of 0 "pair_count" in computeBracketPairs().

[Aric Stewart]

Signed-off-by: Aric Stewart

On 2/18/18 12:08 PM, Henri Verbeet wrote:
> Signed-off-by: Henri Verbeet
>
> dlls/usp10/bidi.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/usp10/bidi.c b/dlls/usp10/bidi.c
> index d94dc7b..8f94ede 100644
> --- a/dlls/usp10/bidi.c
> +++ b/dlls/usp10/bidi.c
> @@ -741,6 +741,10 @@ static BracketPair *computeBracketPairs(IsolatedRun *iso_run)
>               }
>           }
>       }
> -
> - heap_free(open_stack);
> - heap_free(stack_index);
> -
> if (pair_count == 0)
> {
>     heap_free(out);
>
> @@ -749,8 +753,6 @@ static BracketPair *computeBracketPairs(IsolatedRun *iso_run)
>       else if (pair_count > 1)
>           qsort(out, pair_count, sizeof(BracketPair), compr);
> - heap_free(open_stack);
> - heap_free(stack_index);
>  return out;
> }