On 10/18/2018 3:09 PM, Paul B Mahol wrote:
> + if (!phase1) {
> + begin = 0;
> + } else if (phase1 == 1) {
> + begin = peak - *len / 2;
> + } else {
> + begin = (.997f - (2 - phase1) * .22f) * *len + .5f;
> + end = (.997f + (0 - phase1) * .22f) * *len + .5f;
> + begin = peak - (begin & ~3);
> + end = peak + 1 + ((end + 3) & ~3);
> + *len = end - begin;
> + *h = av_realloc(*h, *len * sizeof(**h));
Either do
*h = av_realloc_f(*h, *len, sizeof(**h));
Or use a temp variable, then manually free *h on failure. Otherwise
you're leaking the original buffer if it failed to be reallocated.
> + if (!*h) {
> + av_free(pi_wraps);
> + av_free(work);
> + return AVERROR(ENOMEM);
> + }
> + }
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
