[Bug 761916] New: gstinfo: Make GST_DEBUG_PAD_NAME "MT crash safe" (for gcc)

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Feb 12 10:34:01 UTC 2016


https://bugzilla.gnome.org/show_bug.cgi?id=761916

            Bug ID: 761916
           Summary: gstinfo: Make GST_DEBUG_PAD_NAME "MT crash safe" (for
                    gcc)
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: All
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: havard.graff at gmail.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
                CC: gstreamer at pexip.com
     GNOME version: ---

Created attachment 320952
  --> https://bugzilla.gnome.org/attachment.cgi?id=320952&action=edit
test and fix

The pad may be unparented while this macro is called which could result
in a segfault. The new macro protects against this. The parent may still
be disposed while the macro is called, but this will not result in a
crash (but the parent name may be garbage). Using gst_pad_get_parent ()
is undesirable since it takes the object lock.

The patch take advantage of compound expressions available as a C
extension in GCC and some other compilers.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list