Michael Niedermayer:
> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
> ---
> libavfilter/signature_lookup.c | 21 ++++++++++-----------
> 1 file changed, 10 insertions(+), 11 deletions(-)
>
> diff --git a/libavfilter/signature_lookup.c b/libavfilter/signature_lookup.c
> index 86dd0c66754..52a97e1bc7e 100644
> --- a/libavfilter/signature_lookup.c
> +++ b/libavfilter/signature_lookup.c
> @@ -37,6 +37,15 @@
> #define STATUS_END_REACHED 1
> #define STATUS_BEGIN_REACHED 2
>
> +static void sll_free(MatchingInfo **sll)
> +{
> + while (*sll) {
> + MatchingInfo *tmp = *sll;
> + *sll = (*sll)->next;
> + av_free(tmp);
> + }
This does not clear the pointers at all. This does (and avoids
indirections).
static void sll_free(MatchingInfo **sllp)
{
MatchingInfo *sll = *sllp;
*sllp = NULL;
while (sll) {
MatchingInfo *tmp = sll;
sll = sll->next;
av_free(tmp);
}
}
> +}
> +
> static void fill_l1distlut(uint8_t lut[])
> {
> int i, j, tmp_i, tmp_j,count;
> @@ -520,16 +529,6 @@ static MatchingInfo evaluate_parameters(AVFilterContext
> *ctx, SignatureContext *
> return bestmatch;
> }
>
> -static void sll_free(MatchingInfo *sll)
> -{
> - void *tmp;
> - while (sll) {
> - tmp = sll;
> - sll = sll->next;
> - av_freep(&tmp);
> - }
> -}
> -
> static MatchingInfo lookup_signatures(AVFilterContext *ctx, SignatureContext
> *sc, StreamContext *first, StreamContext *second, int mode)
> {
> CoarseSignature *cs, *cs2;
> @@ -572,7 +571,7 @@ static MatchingInfo lookup_signatures(AVFilterContext
> *ctx, SignatureContext *sc
> "ratio %f, offset %d, score %d, %d frames matching\n",
> bestmatch.first->index, bestmatch.second->index,
> bestmatch.framerateratio, bestmatch.offset,
> bestmatch.score, bestmatch.matchframes);
> - sll_free(infos);
> + sll_free(&infos);
> }
> } while (find_next_coarsecandidate(sc, second->coarsesiglist, &cs, &cs2,
> 0) && !bestmatch.whole);
> return bestmatch;
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
