[Bug 707523] d3dvideosink memleak

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=707523
  GStreamer | gst-plugins-bad | 1.x

--- Comment #6 from Nicola <lists at svrinformatica.it> 2013-09-20 17:42:52 UTC ---
I'm a bit busy now, later or tomorrow I'll sent a git patch against gst-sdk and
1.x, basically the problem is here:

http://cgit.freedesktop.org/gstreamer-sdk/gst-plugins-bad/tree/sys/d3dvideosink/d3dvideosink.c?h=sdk-0.10.23#n2354

if you have more than a viewer the code at line 2356-2358 is not reached, so
these lines:

GST_D3DVIDEOSINK_D3D_DEVICE_LOCK (sink);
gst_d3dvideosink_release_d3d_device (sink);
GST_D3DVIDEOSINK_D3D_DEVICE_UNLOCK (sink);

need to be after:

shared.element_count--;

and however before the goto

if you have only one viewer the problem does not happen (shared.element_count
is never >0)

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