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 332796 - [asfdemux] cannot play an ASF stream
[asfdemux] cannot play an ASF stream
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
git master
Other All
: Normal normal
: 0.10.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-02-27 23:30 UTC by Fabrizio Gennari
Modified: 2006-03-01 10:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for ASF support (876 bytes, patch)
2006-02-28 18:59 UTC, Fabrizio Gennari
committed Details | Review

Description Fabrizio Gennari 2006-02-27 23:30:54 UTC
Please describe the problem:
I downloaded an edition of the Dutch radio journal
(http://www.nos.nl/nosjournaal/archief/index.html) with a simple pipeline

gstreamer/tools/.libs/gst-launch-0.10 mmssrc
location="mms://streams.omroep.nl/nos/nieuws/2006/februari/audio/260206/Radio 1
Journaal       1200-1300     - zo 26 feb 2006 - 12 00_20k.wma" ! filesink
location=nos.wma

I can play the file with this pipeline
~/gstreamer/tools/.libs/gst-launch-0.10 filesrc location=~/nos.wma ! ffdemux_asf
.audio_00 ! ffdec_wmav2 ! alsasink

But this pipeline fails (immediately exits)
~/gstreamer/tools/.libs/gst-launch-0.10 filesrc location=~/nos.wma ! asfdemux
.audio_00 ! ffdec_wmav2 ! alsasink

There is probably something wrong with asfdemux. Increasing the debug level, the
output looks suspicious to me (this line is output once per packet)
asfdemux(18419) gstasfdemux.c(2219):gst_asf_demux_handle_data: multiple = 0,
sequence = 0, padsize = 119, packet length = 6
6 bytes for the packet length seems way too short


Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Fabrizio Gennari 2006-02-28 18:59:53 UTC
Created attachment 60340 [details] [review]
Patch for ASF support

I don't know why packet length was read after sequence and padsize. In versions of asfdemux.c until 1.91 it was read first. In ffmpeg's libavformat/asf.c it is read first (see function asf_get_packet). And this patch makes the above-mentioned file work.

Sorry, I cannot attach it, it is 9MB long. I can send via e-mail if needed. However, you can download it using the mentioned procedure
Comment 2 Tim-Philipp Müller 2006-03-01 10:04:26 UTC
No idea why I swapped the order when porting, probably just a silly copy'n'paste-o. Thanks for the patch!

 2006-03-01  Tim-Philipp Müller  <tim at centricular dot net>

         * gst/asfdemux/gstasfdemux.c: (gst_asf_demux_handle_data):
           Read packet size, sequence and padsize in right order again
           (fixes #332796; patch by: Fabrizio Gennari).