va: Make the caps pointer operation atomic in vadecoder.
authorHe Junyan <junyan.he@intel.com>
Fri, 15 Jan 2021 08:05:06 +0000 (16:05 +0800)
committerHe Junyan <junyan.he@intel.com>
Fri, 15 Jan 2021 12:40:21 +0000 (20:40 +0800)
The vadecoder's srcpad_caps and sinkpad_caps pointers are outside of the
mutex protection. Just make all operation for them atomic.

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

sys/va/gstvadecoder.c

index 45be8a0..93fcb33 100644 (file)
@@ -373,7 +373,7 @@ gst_va_decoder_get_srcpad_caps (GstVaDecoder * self)
 
   g_return_val_if_fail (GST_IS_VA_DECODER (self), FALSE);
 
-  if (self->srcpad_caps)
+  if (g_atomic_pointer_get (&self->srcpad_caps))
     return gst_caps_ref (self->srcpad_caps);
 
   if (_get_codec_caps (self))
@@ -396,7 +396,7 @@ gst_va_decoder_get_sinkpad_caps (GstVaDecoder * self)
 {
   g_return_val_if_fail (GST_IS_VA_DECODER (self), FALSE);
 
-  if (self->sinkpad_caps)
+  if (g_atomic_pointer_get (&self->sinkpad_caps))
     return gst_caps_ref (self->sinkpad_caps);
 
   if (_get_codec_caps (self))