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 339041 - qtdemux : Floating point exception
qtdemux : Floating point exception
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-04-19 15:20 UTC by j^
Modified: 2006-05-03 21:58 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
check for 0 and add qtdrw (1.33 KB, patch)
2006-04-19 15:23 UTC, j^ 		committed Details | Review

Description j^ 2006-04-19 15:20:14 UTC 
trying to open http://nujus.net/gh/rantapod/rantapod_018_ref.mov
i get a Fixedpoint exception in qtdemux.

looking at the code its 
 stream->samples_per_frame * stream->bytes_per_frame beeing 0.

attached is a patch that checks for 0, not sure if the file works that way,
since this file has a so far unknown header of qdrw, Apple Quick Draw,
the patch also adds this to the list and sets video/x-qdrw.

i will also file a patch against gst-ffmpeg to enable the qdrv decoder.
Comment 1 j^ 2006-04-19 15:23:21 UTC 
Created attachment 63876 [details] [review]
check for 0 and add qtdrw
Comment 2 Wim Taymans 2006-04-20 09:48:17 UTC 
the real problem for the floating point error is that a video track should nt be treated as an audio track when collecting the samples. Following commit fixes the floating point error and adds the qtdrw mime-type.

        Patch by: j^ <j at bootlab dot org>

        * gst/qtdemux/qtdemux.c: (qtdemux_parse_trak),
        (qtdemux_video_caps):
        Never treat video streams as an audio stream.
        Add qtdrw mime type.
        Fixes #339041