diff options
author | Tim-Philipp Müller <tim.muller@collabora.co.uk> | 2013-02-18 16:32:13 +0000 |
---|---|---|
committer | Tim-Philipp Müller <tim.muller@collabora.co.uk> | 2013-02-18 16:35:08 +0000 |
commit | 34b81f7c93fd09db77ebb58301de5ef967c15157 (patch) | |
tree | 4f672f8391dc2140fbaeff82a830ea5d2474ad8a | |
parent | 048d256041a68d0f4e64ddbb73a6bcf325e6ff17 (diff) |
qtdemux: fix potential crash on short MOOV atom
Don't unmap short MOOV atom buffer twice, which happened
in the case where we don't fix up the MOOV atom.
Fixes crashes when thumbnailing partial mp4 file where
the MOOV atom is still incomplete.
https://bugzilla.gnome.org/show_bug.cgi?id=694010
-rw-r--r-- | gst/isomp4/qtdemux.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/gst/isomp4/qtdemux.c b/gst/isomp4/qtdemux.c index 80983618e..21f1efbba 100644 --- a/gst/isomp4/qtdemux.c +++ b/gst/isomp4/qtdemux.c @@ -2751,6 +2751,7 @@ gst_qtdemux_loop_state_header (GstQTDemux * qtdemux) if (ret != GST_FLOW_OK) goto beach; gst_buffer_map (moov, &map, GST_MAP_READ); + if (length != map.size) { /* Some files have a 'moov' atom at the end of the file which contains * a terminal 'free' atom where the body of the atom is missing. @@ -2760,13 +2761,14 @@ gst_qtdemux_loop_state_header (GstQTDemux * qtdemux) guint8 *final_data = map.data + (map.size - 8); guint32 final_length = QT_UINT32 (final_data); guint32 final_fourcc = QT_FOURCC (final_data + 4); - gst_buffer_unmap (moov, &map); + if (final_fourcc == FOURCC_free && map.size + final_length - 8 == length) { /* Ok, we've found that special case. Allocate a new buffer with * that free atom actually present. */ GstBuffer *newmoov = gst_buffer_new_and_alloc (length); gst_buffer_copy_into (newmoov, moov, 0, 0, map.size); + gst_buffer_unmap (moov, &map); gst_buffer_map (newmoov, &map, GST_MAP_WRITE); memset (map.data + length - final_length + 8, 0, final_length - 8); gst_buffer_unref (moov); |