[Bug 736986] qtdemux should handle AAC audio without ESDS atom

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Sep 22 08:24:47 PDT 2014


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

Thiago Sousa Santos <thiagossantos> changed:

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

--- Comment #4 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-09-22 15:24:45 UTC ---
Review of attachment 286656:
 --> (https://bugzilla.gnome.org/review?bug=736986&attachment=286656)

::: gst/isomp4/qtdemux.c
@@ +8700,3 @@
+          gint len = QT_UINT32 (stsd_data);
+
+          if (len > 50) {

Shouldn't it be >= ?

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


More information about the gstreamer-bugs mailing list