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 336904 - Problem playing some AVI file when splitting large chunks of audio data
Problem playing some AVI file when splitting large chunks of audio data
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-04-02 11:00 UTC by Fabrizio Gennari
Modified: 2006-04-21 18:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to fix chunk size (577 bytes, patch)
2006-04-02 11:00 UTC, Fabrizio Gennari
committed Details | Review

Description Fabrizio Gennari 2006-04-02 11:00:07 UTC
Please describe the problem:
In function gst_avi_demux_massage_index, if a very large chunk of data is found,
it is split in smaller chunks, and each chunk is indexed separately.

When splitting audio chunks, the block alignment is not taken in consideration,
so the smaller chunks could be of size which is not a multiple of the block
alignment.

This causes problems later, when passing the buffer to the next element,
negotiation could fail with a wrong size error.

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Fabrizio Gennari 2006-04-02 11:00:54 UTC
Created attachment 62587 [details] [review]
Patch to fix chunk size
Comment 2 Wim Taymans 2006-04-21 18:03:44 UTC
        Patch by: Fabrizio Gennari <fabrizio dot ge at tiscali dot it>

        * gst/avi/gstavidemux.c: (gst_avi_demux_parse_stream),
        (gst_avi_demux_parse_index), (gst_avi_demux_massage_index):
        When splitting audio chunks, the block alignment is not taken in
        consideration, so the smaller chunks could be of size which is
        not a multiple of the block alignment. Fixes #336904