Michael Niedermayer <mich...@niedermayer.cc> writes:
> doesnt the failure path here need the free too ?
> int ret = fir_to_phase(s, &h[longer], &n, &post_peak, s->phase);
> if (ret < 0)
> return ret;
>
I checked fir_to_phase() and found that the only place it allocates memory
in
h[longer] is vis
*h = av_realloc_f();
If fir_to_phase() failed after this allocations, then *h must be NULL,
which implies that av_realloc_f() failed. In that case, we don't need to
av_free(h[longer]). However, I still need to double check other allocation
site
in case we need to free more memory.
Thanks,
Lidong
_______________________________________________
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".
