msdkdec: avoid early destruction of frame in dynamic resolution change

In cases where we do hard resest, the current code destroys the frame
which has new resolution bit early and this causes buffer_unmap
warnings. Keep an extra ref to the frame internally to avoid this.

1 files changed, 6 insertions, 0 deletions

diff --git a/sys/msdk/gstmsdkdec.c b/sys/msdk/gstmsdkdec.c
index a9a82c463..393d5a1d0 100644
--- a/sys/msdk/gstmsdkdec.c
+++ b/sys/msdk/gstmsdkdec.c
@@ -922,6 +922,12 @@ gst_msdkdec_handle_frame (GstVideoDecoder * decoder, GstVideoCodecFrame * frame)
     if (thiz->force_reset_on_res_change)
       hard_reset = TRUE;
 
+    /* Config changed dynamically and we are going to do a full reset,
+     * this will unref the input frame which has the new configuration.
+     * Keep a ref to the input_frame to keep it alive */
+    if (thiz->initialized && thiz->do_renego)
+      gst_video_codec_frame_ref (frame);
+
     gst_msdkdec_negotiate (thiz, hard_reset);
   }