[Bug 760821] tracerrecord: Fix memory leaks and mishandlings
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Wed Jan 20 16:58:16 PST 2016
https://bugzilla.gnome.org/show_bug.cgi?id=760821
--- Comment #8 from Vineeth <vineeth.tm at samsung.com> ---
(In reply to Stefan Sauer (gstreamer, gtkdoc dev) from comment #7)
> For the structure, I'd like to avoid the need to manually free them. This
> makes the code like here:
> http://cgit.freedesktop.org/gstreamer/gstreamer/tree/plugins/tracers/
> gstlatency.c#n204
> quite ugly.
>
> We have a simillar case in:
> http://cgit.freedesktop.org/gstreamer/gst-plugins-base/tree/gst/playback/
> gstdecodebin2.c#n4409
>
> Maybe I should revert
> https://bug760821.bugzilla-attachments.gnome.org/attachment.cgi?id=319322
> and actually use g_value_take_boxed() and when freeing the structure, also
> free all sub structures recursively. Any opinions?
It might work out.. I am not exactly sure how get the memory location of the
sub structures to free the same. Maybe there is a way. Other maintainers should
be able to answer this.
In the meantime i will split the other 2 leaks, so that we can work on this
seperately
--
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