[Bug 736047] qtdemux: check we have a corresponding stream after parsing a tfhd node

[GStreamer (bugzilla.gnome.org)]

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

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #285359|none                        |reviewed
             status|                            |

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-09-04 12:39:40 UTC ---
Review of attachment 285359:
 --> (https://bugzilla.gnome.org/review?bug=736047&attachment=285359)

::: gst/isomp4/qtdemux.c
@@ +2794,3 @@
         qtdemux_tree_get_child_by_type_full (traf_node, FOURCC_tfdt,
         &tfdt_data);
+    if (tfdt_node && stream) {

Or maybe if we have no stream we shouldn't even get to this code here or at
least not ask for the tfdt node?

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