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 701961 - dashdemux: Spurious critical if there is no initialization segment
dashdemux: Spurious critical if there is no initialization segment
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.1.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-06-10 22:02 UTC by Olivier Crête
Modified: 2013-07-02 05:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
dashdemux: Handle case without an initialization segment (2.26 KB, patch)
2013-06-10 22:02 UTC, Olivier Crête
none Details | Review

Description Olivier Crête 2013-06-10 22:02:34 UTC
Created attachment 246450 [details] [review]
dashdemux: Handle case without an initialization segment

I have a stream with no initialisation segment, if should not try to fetch it. Probably gst_dash_demux_get_next_header() should have a different return value if there is no initialisation segment in the mpd? Or maybe there should be a check at every place where need_header is set to true ?
Comment 1 Thiago Sousa Santos 2013-07-02 05:02:10 UTC
Pushed a rebased version.

commit 3f26bb9154134e6b0dd3d80d85ca9cefc7853051
Author: Olivier Crête <olivier.crete@collabora.com>
Date:   Mon Jun 10 18:01:14 2013 -0400

    dashdemux: Handle case without an initialization segment
    
    If no initialization segment is defined, then don't print a
    critical or a warning, just ignore it.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=701961