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 711829 - matroskademux: assert on division by zero
matroskademux: assert on division by zero
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 1.2.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-11-11 10:13 UTC by David Svensson Fors
Modified: 2013-11-26 11:48 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Avoid division by zero assert in gst_matroska_demux_search_pos (1.39 KB, patch)
2013-11-11 10:13 UTC, David Svensson Fors 		committed Details | Review

Description David Svensson Fors 2013-11-11 10:13:43 UTC 
Created attachment 259544 [details] [review]
Avoid division by zero assert in gst_matroska_demux_search_pos

When doing a seek to position 0 when already positioned on 0 in a mkv with no index (still being recorded), there is an assert in _gst_util_uint64_scale for denom being 0.

This is caused by a call from gst_matroska_demux_search_pos where otime – demux->stream_start_time == 0.

I have attached the patch that we are using as workaround.

This condition occurs very often but not always. Sometimes it looks like the position has moved from 0 and the problem does not happen even though the pipeline was paused directly.
Comment 1 Sebastian Dröge (slomo) 2013-11-11 10:31:32 UTC 
commit b3e0b1dbcaa452c5419b3ad254a0f8cca6167f4a
Author: Per x Johansson <perxjoh@axis.com>
Date:   Thu Oct 24 13:16:42 2013 +0200

    matroskademux: Avoid division by zero assert in gst_matroska_demux_search_pos
    
    https://bugzilla.gnome.org/show_bug.cgi?id=711829