[Bug 761916] gstinfo: Make GST_DEBUG_PAD_NAME "MT crash safe"

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Feb 22 23:59:12 UTC 2016


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

--- Comment #10 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Bit undecided what to do with this.

It avoids one problem, but there are more left.

I wonder if it wouldn't make sense to do things more properly here (incurring a
small performance impact), i.e get parent with lock (and a ref then), copy name
into g_alloca()-allocated memory or such (would need a max-name-length
limitation then, which should be ok).

Arguably this is only used when debug logging is actually active, so not in the
case where it's not, and when logging is active a teeny bit of extra overhead
won't matter that much, won't it?

-- 
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