[Bug 701110] videomixer: sinkpads GSList is not protected for multi-threading

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=701110
  GStreamer | gst-plugins-good | git

--- Comment #5 from [laucha] <lauchapc87 at gmail.com> 2014-03-07 13:35:05 UTC ---
Created an attachment (id=271223)
 View: https://bugzilla.gnome.org/attachment.cgi?id=271223
 Review: https://bugzilla.gnome.org/review?bug=701110&attachment=271223

workaround for the segfault and a patch for the freezing bug. Both are for
videomixer

Nicolas, here is the patch. I've tested that works in the following way:

1) If I run the example without the patch it freezes after one hour of running
aproximately. If I run 3  instances of the example at the same time, one of
them freezes after 5 minutes at most.

2) I ran 4 instances of the example with the patch applied and it was running
for 5 days without freezing.

Apparently seems that the patch works. Please confirm if you want me to change
something.

If you want to test the patch, first at all you will have to apply another
patch that is a workaround for the segfault that I describe on comment 2 and 3.
The real patch is for the freezing case.

Until now I've not had the time to investigate why the ptr to the mix function
was seted to NULL (I think it is because a caps's renegotiation when I remove
the pad).

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