[FFmpeg-devel] [PATCH] lavc/videotoolbox: fix failure to decode PAFF

Aman Gupta ffmpeg at tmm1.net
Wed Apr 4 04:24:43 EEST 2018


On Wed, Mar 28, 2018 at 9:19 PM, Rodger Combs <rodger.combs at gmail.com>
wrote:

> ---
>  libavcodec/videotoolbox.c | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/libavcodec/videotoolbox.c b/libavcodec/videotoolbox.c
> index f82c31c5df..57b6698e1b 100644
> --- a/libavcodec/videotoolbox.c
> +++ b/libavcodec/videotoolbox.c
> @@ -326,11 +326,8 @@ static int videotoolbox_set_frame(AVCodecContext
> *avctx, AVFrame *frame)
>
>      CVPixelBufferRef *ref = (CVPixelBufferRef *)frame->buf[0]->data;
>
> -    if (*ref) {
> -        av_log(avctx, AV_LOG_ERROR, "videotoolbox: frame already set?\n");
> -        av_frame_unref(frame);
> -        return AVERROR_EXTERNAL;
> -    }
> +    if (*ref)
> +        CVPixelBufferRelease(*ref);
>
>      *ref = vtctx->frame;
>      vtctx->frame = NULL;
>

LGTM. Do you have a sample PAFF file that now works after this patch?

Aman


> --
> 2.16.2
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list