diff options
author | Haihao Xiang <haihao.xiang@intel.com> | 2020-12-29 09:41:05 +0800 |
---|---|---|
committer | GStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org> | 2020-12-30 11:55:25 +0000 |
commit | 24d1e75be5d963c8c3b817e3458bb4bde31e86aa (patch) | |
tree | 2f0ae9fb4906d2d64cc025bdc519745dc6aae79d | |
parent | 5015a66c561d203e339b4c3a1c24dec5b1f9a6ef (diff) | |
download | gstreamer-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.c | 1 |
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; } |