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 778199 - hlsdemux: Fix segfault during bitrate switch
hlsdemux: Fix segfault during bitrate switch
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-02-05 11:27 UTC by Seungha Yang
Modified: 2018-11-03 14:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
hlsdemux: Fix segfault during bitrate switch (951 bytes, patch)
2017-02-05 11:27 UTC, Seungha Yang
needs-work Details | Review

Description Seungha Yang 2017-02-05 11:27:22 UTC
Target variant stream can be updated in _updtae_playlist()
Comment 1 Seungha Yang 2017-02-05 11:27:55 UTC
Created attachment 344970 [details] [review]
hlsdemux: Fix segfault during bitrate switch
Comment 2 Seungha Yang 2017-02-05 11:36:08 UTC
Since error log includes private uri, I cannot attach it, sorry.

Sometimes, segfault happen at
https://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/ext/hls/gsthlsdemux.c#n1556

The cause is updated current_variant.
Inside of _update_playlist(), demux's current variant can be updated by
https://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/ext/hls/gsthlsdemux.c#n1374
and 
https://cgit.freedesktop.org/gstreamer/gst-plugins-bad/tree/ext/hls/gsthlsdemux.c#n1250
Comment 3 Sebastian Dröge (slomo) 2017-03-02 18:22:35 UTC
Review of attachment 344970 [details] [review]:

This seems like it should be solved at a different place because

::: ext/hls/gsthlsdemux.c
@@ +1568,3 @@
     gchar *uri;
 
+    uri = gst_m3u8_get_uri (demux->current_variant->m3u8);

This seems to defeat the purpose of the code below, where new_variant is updated before going to retry_failover_protection
Comment 4 GStreamer system administrator 2018-11-03 14:04:20 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/516.