On 1/6/2021 12:12 AM, Xu Guangxin wrote:
fixes #8857
If we do not clear the enc_ctrl, we will reuse previous frames' data like
FrameType.
---
libavcodec/qsvenc.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c
index 2bd2a56227..94473c4eab 100644
--- a/libavcodec/qsvenc.c
+++ b/libavcodec/qsvenc.c
@@ -1249,6 +1249,8 @@ static void clear_unused_frames(QSVEncContext *q)
while (cur) {
if (cur->used && !cur->surface.Data.Locked) {
free_encoder_ctrl_payloads(&cur->enc_ctrl);
+ //do not reuse enc_ctrl from previous frame
+ memset(&cur->enc_ctrl, 0, sizeof(cur->enc_ctrl));
I assume cur->enc_ctrl.ExtParam is not set, right? Otherwise this memset
could lead to leaks.
if (cur->frame->format == AV_PIX_FMT_QSV) {
av_frame_unref(cur->frame);
}
_______________________________________________
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".
