It is unnecessary in case of user-supplied frames, because
it happens directly after a av_frame_ref() with the same
src and dst.
Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com>
---
libavcodec/mpegvideo_enc.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 26a596430d..cb6b801b2a 100644
--- a/libavcodec/mpegvideo_enc.c
+++ b/libavcodec/mpegvideo_enc.c
@@ -1168,6 +1168,11 @@ static int load_input_picture(MpegEncContext *s, const
AVFrame *pic_arg)
ret = alloc_picture(s, pic);
if (ret < 0)
return ret;
+ ret = av_frame_copy_props(pic->f, pic_arg);
+ if (ret < 0) {
+ ff_mpeg_unref_picture(s->avctx, pic);
+ return ret;
+ }
for (int i = 0; i < 3; i++) {
int src_stride = pic_arg->linesize[i];
@@ -1209,11 +1214,6 @@ static int load_input_picture(MpegEncContext *s, const
AVFrame *pic_arg)
}
emms_c();
}
- ret = av_frame_copy_props(pic->f, pic_arg);
- if (ret < 0) {
- ff_mpeg_unref_picture(s->avctx, pic);
- return ret;
- }
pic->display_picture_number = display_picture_number;
pic->f->pts = pts; // we set this here to avoid modifying pic_arg
--
2.34.1
_______________________________________________
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".
