From: "Eoff, Ullysses A" <ullysses.a.eoff-at-intel.com@ffmpeg.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 1/2] lavfi/qsvpp: fix after 85c938fa28
Date: Tue, 8 Feb 2022 22:53:32 +0000
Message-ID: <DM6PR11MB27462034A0D19073333C303ABE2D9@DM6PR11MB2746.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20220208182639.25696-1-anton@khirnov.net>
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Anton Khirnov
> Sent: Tuesday, February 8, 2022 10:27 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH 1/2] lavfi/qsvpp: fix after 85c938fa28
>
> ---
> libavfilter/qsvvpp.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/libavfilter/qsvvpp.c b/libavfilter/qsvvpp.c
> index 35769dfd60..954f882637 100644
> --- a/libavfilter/qsvvpp.c
> +++ b/libavfilter/qsvvpp.c
> @@ -796,7 +796,7 @@ int ff_qsvvpp_filter_frame(QSVVPPContext *s, AVFilterLink *inlink, AVFrame *picr
> AVFilterLink *outlink = ctx->outputs[0];
> QSVAsyncFrame aframe;
> mfxSyncPoint sync;
> - QSVFrame *in_frame, *out_frame, *tmp;
> + QSVFrame *in_frame, *out_frame;
> int ret, filter_ret;
>
> while (s->eof && av_fifo_read(s->async_fifo, &aframe, 1) >= 0) {
> @@ -857,15 +857,15 @@ int ff_qsvvpp_filter_frame(QSVVPPContext *s, AVFilterLink *inlink, AVFrame *picr
> ret = MFXVideoCORE_SyncOperation(s->session, aframe.sync, 1000);
> } while (ret == MFX_WRN_IN_EXECUTION);
>
> - filter_ret = s->filter_frame(outlink, tmp->frame);
> + filter_ret = s->filter_frame(outlink, aframe.frame->frame);
> if (filter_ret < 0) {
> - av_frame_free(&tmp->frame);
> + av_frame_free(&aframe.frame->frame);
> return filter_ret;
> }
>
> - tmp->queued--;
> + aframe.frame->queued--;
> s->got_frame = 1;
> - tmp->frame = NULL;
> + aframe.frame->frame = NULL;
> }
> } while(ret == MFX_ERR_MORE_SURFACE);
>
> --
> 2.34.1
>
Fixes https://trac.ffmpeg.org/ticket/9629 for me.
LGTM
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
prev parent reply other threads:[~2022-02-08 22:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-08 18:26 Anton Khirnov
2022-02-08 18:26 ` [FFmpeg-devel] [PATCH 2/2] lavu/hwcontext_qsv: fix a potential infinite loop Anton Khirnov
2022-02-08 19:31 ` James Almer
2022-02-08 22:53 ` Eoff, Ullysses A [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DM6PR11MB27462034A0D19073333C303ABE2D9@DM6PR11MB2746.namprd11.prod.outlook.com \
--to=ullysses.a.eoff-at-intel.com@ffmpeg.org \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git