[Bug 728490] v4l2src: Missing unref in decide_allocation

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Apr 18 07:44:35 PDT 2014


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

Nicolas Dufresne <nicolas.dufresne> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |nicolas.dufresne at collabora.
                   |                            |co.uk
     Ever Confirmed|0                           |1

--- Comment #2 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2014-04-18 14:44:30 UTC ---
Ok, gst_query_set_nth_allocation_pool() and gst_query_add_allocation_pool()
will ref the given pool, so the pool is transferred none.

gst_query_parse_nth_allocation_pool() give you a ref on the pool. This mean
that if you didn't switch the pool to our own pool, you would be leaking a
downstream pool. Here's what I propose to fix this correctly:

after set_nth.. and add_allo, unref the pool. When replacing the pool, unref
the pool from the allocation, and ref our own pool. This should be clean.

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