[Bug 667419] matroskamux memleaks

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Jan 10 05:00:39 PST 2012


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

--- Comment #9 from Mark Nauwelaerts <mnauw at users.sourceforge.net> 2012-01-10 13:00:35 UTC ---
Maybe stating the obvious, but the allocation happening more than once is
likely due to an upstream video parser that incrementally parses more info
leading to new caps leading to renegotiation in the muxer's _set_caps (which is
so far not expecting this and can lead to leaking all sorts of context stuff,
codec string, codec-data etc).

AFAIK most of the other code in matroskamux should already take care to free if
it decides to replace some context field (?)

In particular, would be nice if one can come up with a clever trick/hack to
handle it safely for all the possible fields of the context (audio and/or
video) at maybe the start of _set_caps, then it does not have to be distributed
all over each individual assignment ...

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