After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 746803 - mxfdemux: fix segment->delta_entries memory leak
mxfdemux: fix segment->delta_entries memory leak
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
unspecified
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-03-26 10:52 UTC by Guillaume Desmottes
Modified: 2015-04-04 18:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
mfdemux: reset the segment before freeing it (1.09 KB, patch)
2015-03-26 10:53 UTC, Guillaume Desmottes 		committed Details | Review

Description Guillaume Desmottes 2015-03-26 10:52:19 UTC 
.
Comment 1 Guillaume Desmottes 2015-03-26 10:53:47 UTC 
Created attachment 300347 [details] [review]
mfdemux: reset the segment before freeing it

mxf_index_table_segment_parse() allocates data inside the segment (like
segment->delta_entries) which have to be freeded using
mxf_index_table_segment_reset().
Comment 2 Tim-Philipp Müller 2015-04-04 18:57:01 UTC 
Pushed, thanks:

commit 4d790dae67f5c80827abca8e978352a71f03972b
Author: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Date:   Thu Mar 26 11:52:33 2015 +0100

    mxfdemux: fix memory leak
    
    Reset the internal segment before freeing it.
    
    mxf_index_table_segment_parse() allocates data inside the segment
    (like segment->delta_entries) which have to be freed using
    mxf_index_table_segment_reset().
    
    https://bugzilla.gnome.org/show_bug.cgi?id=746803