On Wed, Mar 28, 2018 at 9:19 PM, Rodger Combs <rodger.co...@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@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
