[FFmpeg-devel] [PATCH 4/4] avfilter/vf_zscale: realign output buffer if needed
Marton Balint
cus at passwd.hu
Mon Mar 14 23:06:03 EET 2022
Output buffer alignment might be different to ZIMG_ALIGNMENT or it may not be
aligned at all if a downstream filter (e.g. vf_pad) intentionally misaligns it.
Or maybe we should unconditionally always allocate output with
av_frame_get_buffer() instead of ff_get_video_buffer()?
Signed-off-by: Marton Balint <cus at passwd.hu>
---
libavfilter/vf_zscale.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_zscale.c b/libavfilter/vf_zscale.c
index ceefc95224..2061e38bcc 100644
--- a/libavfilter/vf_zscale.c
+++ b/libavfilter/vf_zscale.c
@@ -632,7 +632,7 @@ static int graphs_build(AVFrame *in, AVFrame *out, const AVPixFmtDescriptor *des
return 0;
}
-static int realign_frame(const AVPixFmtDescriptor *desc, AVFrame **frame)
+static int realign_frame(const AVPixFmtDescriptor *desc, AVFrame **frame, int needs_copy)
{
AVFrame *aligned = NULL;
int ret = 0, plane, planes;
@@ -654,10 +654,10 @@ static int realign_frame(const AVPixFmtDescriptor *desc, AVFrame **frame)
if ((ret = av_frame_get_buffer(aligned, ZIMG_ALIGNMENT)) < 0)
goto fail;
- if ((ret = av_frame_copy(aligned, *frame)) < 0)
+ if (needs_copy && (ret = av_frame_copy(aligned, *frame)) < 0)
goto fail;
- if ((ret = av_frame_copy_props(aligned, *frame)) < 0)
+ if (needs_copy && (ret = av_frame_copy_props(aligned, *frame)) < 0)
goto fail;
av_frame_free(frame);
@@ -786,9 +786,12 @@ static int filter_frame(AVFilterLink *link, AVFrame *in)
goto fail;
}
+ if ((ret = realign_frame(odesc, &out, 0)) < 0)
+ goto fail;
+
av_frame_copy_props(out, in);
- if ((ret = realign_frame(desc, &in)) < 0)
+ if ((ret = realign_frame(desc, &in, 1)) < 0)
goto fail;
snprintf(buf, sizeof(buf)-1, "%d", outlink->w);
--
2.31.1
More information about the ffmpeg-devel
mailing list