summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHaihao Xiang <haihao.xiang@intel.com>2020-12-29 09:41:05 +0800
committerGStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org>2020-12-30 11:55:25 +0000
commit24d1e75be5d963c8c3b817e3458bb4bde31e86aa (patch)
tree2f0ae9fb4906d2d64cc025bdc519745dc6aae79d
parent5015a66c561d203e339b4c3a1c24dec5b1f9a6ef (diff)
downloadgstreamer-plugins-bad-24d1e75be5d963c8c3b817e3458bb4bde31e86aa.tar.gz
msdk: needn't close mfx session when failed
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/1927>
-rw-r--r--sys/msdk/gstmsdkcontext.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/sys/msdk/gstmsdkcontext.c b/sys/msdk/gstmsdkcontext.c
index d9210780a..012efe615 100644
--- a/sys/msdk/gstmsdkcontext.c
+++ b/sys/msdk/gstmsdkcontext.c
@@ -207,7 +207,6 @@ gst_msdk_context_open (GstMsdkContext * context, gboolean hardware,
return TRUE;
failed:
- msdk_close_session (priv->session);
return FALSE;
}