msdk: needn't close mfx session when failed
authorHaihao Xiang <haihao.xiang@intel.com>
Tue, 29 Dec 2020 01:41:05 +0000 (09:41 +0800)
committerHaihao Xiang <haihao.xiang@intel.com>
Wed, 30 Dec 2020 05:08:52 +0000 (05:08 +0000)
Otherwise we will get double free issue because mfx session is closed in
finalize. See
https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1867#note_739346
for the double free issue.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/1916>

sys/msdk/gstmsdkcontext.c

index d921078..012efe6 100644 (file)
@@ -207,7 +207,6 @@ gst_msdk_context_open (GstMsdkContext * context, gboolean hardware,
   return TRUE;
 
 failed:
-  msdk_close_session (priv->session);
   return FALSE;
 }