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 542410 - qtdemux not recognizing pcm 8bit streams correctly
qtdemux not recognizing pcm 8bit streams correctly
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.9
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-07-10 18:31 UTC by Thiago Sousa Santos
Modified: 2008-07-10 20:50 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22

Attachments
fixes it. (1.09 KB, patch)
2008-07-10 19:04 UTC, Thiago Sousa Santos 		committed Details | Review

Description Thiago Sousa Santos 2008-07-10 18:31:06 UTC 
Please describe the problem:
When streaming a qt file with 8bit pcm audio (signed=true and with audio fourcc code 'twos') through qtdemux, it recognizes it as pcm 16bits, messing the audio output

Steps to reproduce:
1. Get an qt file with pcm signed 8bit audio and 'twos' fourcc code for audio (this can be done using qtmux element, for instance)
2. Try to demux and play the qt file
3. 


Actual results:
The audio output is wrong

Expected results:
The qtdemux should recognize it as an 8bit pcm stream

Does this happen every time?
Yes

Other information:
Comment 1 Thiago Sousa Santos 2008-07-10 18:58:06 UTC 
The same file plays on mplayer and vlc correctly. Also qtinfo from quicktime-utils package shows that it is a 8bit pcm audio.
Comment 2 Thiago Sousa Santos 2008-07-10 19:04:13 UTC 
Created attachment 114345 [details] [review]
fixes it.

This patch fixes it (according to qtff spec).
Comment 3 Mark Nauwelaerts 2008-07-10 20:50:30 UTC 
Thanks!

2008-07-10  Mark Nauwelaerts  <mark.nauwelaerts@collabora.co.uk>

	Patch by: Thiago Sousa Santos <thiagoss at lcc dot ufcg dot edu dot br>

	* gst/qtdemux/qtdemux.c: (qtdemux_audio_caps):
	Correctly distinguish 8bit vs 16bit raw audio.  Fixes #542410.