[gstreamer-bugs] [Bug 624338] [pulsesink] double-free exception
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Mon Aug 16 02:00:16 PDT 2010
https://bugzilla.gnome.org/show_bug.cgi?id=624338
GStreamer | gst-plugins-good | git
--- Comment #16 from Philippe Normand <phil at base-art.net> 2010-08-16 09:00:11 UTC ---
(In reply to comment #15)
> Looks good in general but you should protected the hash table with a mutex...
> everywhere.
>
Right, I did it only at one place, sorry ;)
> Then it might make sense to add a property to set the pulse client name. E.g.
> in webkit you might want to have all pulsesinks of the same tab in the same
> pulse client but not all pulsesinks of all tabs.
>
Good idea indeed, could that go in a separate bug/patch? I'd be fine with doing
it here but the current patch is already big I think.
> Also, gst_pulse_client_name() should maybe be improved and return something
> like "GStreamer ($PID)" when the other two ways to get a name fail.
Ok, will do that in next iteration of the patch.
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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