av_frame_copy doesn't copy the input's PTS property, which resulted
in the frei0r filter always receiving the same (non-sensical) time.
For example, this results in a static distortion, when it should be
changing over time:
ffmpeg -filter_complex "testsrc2=s=342x142:d=5,frei0r=distort0r" out.mp4
Signed-off-by: Stefan Breunig <stefan-ffmpeg-de...@breunig.xyz>
---
libavfilter/vf_frei0r.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavfilter/vf_frei0r.c b/libavfilter/vf_frei0r.c
index 65ee396bcd..0dd44c4519 100644
--- a/libavfilter/vf_frei0r.c
+++ b/libavfilter/vf_frei0r.c
@@ -361,6 +361,7 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
{
Frei0rContext *s = inlink->dst->priv;
AVFilterLink *outlink = inlink->dst->outputs[0];
+ double time;
/* align parameter is the line alignment, not the buffer alignment.
* frei0r expects line size to be width*4 so we want an align of 1
* to ensure lines aren't padded out. */
@@ -370,6 +371,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
av_frame_copy_props(out, in);
+ time = in->pts * av_q2d(inlink->time_base);
+
if (in->linesize[0] != out->linesize[0]) {
AVFrame *in2 = ff_default_get_video_buffer2(outlink, outlink->w,
outlink->h, 1);
if (!in2)
@@ -379,7 +382,7 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
in = in2;
}
- s->update(s->instance, in->pts * av_q2d(inlink->time_base),
+ s->update(s->instance, time,
(const uint32_t *)in->data[0],
(uint32_t *)out->data[0]);
--
2.45.2
_______________________________________________
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".
