[gstreamer-bugs] [Bug 631853] [queue2] deadlock when using temp-location and dispatch-properties

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=631853
  GStreamer | gstreamer (core) | unspecified

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at circular-chaos.org

--- Comment #2 from Sebastian Dröge <slomo at circular-chaos.org> 2010-10-11 08:30:00 UTC ---
Calling g_object_notify() with the same lock that is used to protect
get/set_property is an interesting idea :)

In gst_queue2_src_activate_pull() the _open_temp_location_file() is called
without the queue2 mutex, maybe it's ok if the mutex is taken after opening the
temp file? If not, gst_queue2_src_activate_pull() has to be changed too. If the
mutex is really necessary here (I don't think so, all accesses to the temp file
should be protected by other mutexes already) open_temp_location() could return
some boolean in an out parameter to have g_object_notify() called outside the
function.

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