FaSTTUBe/GST-Tensordecoder-ov_ep
37
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

mxf: fix descriptor leak

Browse Source 
Free the existing descriptor array, if any, before replacing it.

Fix leaks with the
validate.file.playback.scrub_forward_seeking.test-mpeg2-mp3_mxf scenario.

https://bugzilla.gnome.org/show_bug.cgi?id=748580
This commit is contained in:
Guillaume Desmottes 2015-04-28 11:28:39 +02:00 committed by Thiago Santos
parent 94446498de
commit da113b5ad7
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gst/mxf/mxfmetadata.c
View File

@ -2239,6 +2239,7 @@ mxf_metadata_source_package_resolve (MXFMetadataBase * m, GHashTable * metadata)
if (d->linked_track_id == package->tracks[i]->track_id || if (d->linked_track_id == package->tracks[i]->track_id ||
(d->linked_track_id == 0 && package->n_essence_tracks == 1 && (d->linked_track_id == 0 && package->n_essence_tracks == 1 &&
(package->tracks[i]->type & 0xf0) == 0x30)) { (package->tracks[i]->type & 0xf0) == 0x30)) {
g_free (package->tracks[i]->descriptor);
package->tracks[i]->descriptor = package->tracks[i]->descriptor =
g_new0 (MXFMetadataFileDescriptor *, 1); g_new0 (MXFMetadataFileDescriptor *, 1);
package->tracks[i]->descriptor[0] = d; package->tracks[i]->descriptor[0] = d;
@ -2264,6 +2265,7 @@ mxf_metadata_source_package_resolve (MXFMetadataBase * m, GHashTable * metadata)
n_descriptor++; n_descriptor++;
} }
g_free (package->tracks[i]->descriptor);
package->tracks[i]->descriptor = package->tracks[i]->descriptor =
g_new0 (MXFMetadataFileDescriptor *, n_descriptor); g_new0 (MXFMetadataFileDescriptor *, n_descriptor);
package->tracks[i]->n_descriptor = n_descriptor; package->tracks[i]->n_descriptor = n_descriptor;