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 141554 - vorbis in matroska files makes the vorbisdec plugin segfault
vorbis in matroska files makes the vorbisdec plugin segfault
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
git master
Other Linux
: Normal normal
: 0.8.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-05-01 10:42 UTC by Arwed v. Merkatz
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
test file, matroska with one vorbis audio stream (513.65 KB, application/octet-stream)
2004-05-01 10:51 UTC, Arwed v. Merkatz 		  Details
patch to make demuxing of vorbis in matroska work (1.50 KB, patch)
2004-07-30 17:41 UTC, Arwed v. Merkatz 		none Details | Review

Description Arwed v. Merkatz 2004-05-01 10:42:20 UTC 
$ gst-launch-0.8 filesrc location="test.mka" ! matroskademux ! vorbisdec !
audioconvert ! audioscale ! audio/x-raw-int,rate=48000 ! alsasink device=dsp1
--gst-scheduler=basicgthread
RUNNING pipeline ...
Segmentation fault (core dumped)

+ Trace 46562

  • #0 vorbis_synthesis
    from /usr/lib/libvorbis.so.0
  • #0 vorbis_synthesis
    from /usr/lib/libvorbis.so.0
  • #1 vorbis_dec_chain
    from /usr/lib/gstreamer-0.8/libgstvorbis.so
  • #2 gst_basic_scheduler_chain_wrapper
    at gstbasicscheduler.c line 365
  • #3 run_new_thread
    at gthread-cothreads.h line 138
  • #4 g_thread_create_proxy
    from /usr/lib/libglib-2.0.so.0
  • #5 start_thread
    from /lib/libpthread.so.0
  • #6 clone
    from /lib/libc.so.6 






Comment 1


Arwed v. Merkatz





          2004-05-01 10:51:04 UTC
        



Created attachment 27261 [details]
test file, matroska with one vorbis audio stream






Comment 2


Ronald Bultje





          2004-05-01 18:01:35 UTC
        



We need to gst_pad_push() the codecprivatedata before anything else from the
vorbis stream, it contains the first three vorbis packets.






Comment 3


Benjamin Otte (Company)





          2004-05-04 16:44:47 UTC
        



could you fix that for now by not claiming you're sending audio/x-vorbis ?
I'll look at the segfault.






Comment 4


Benjamin Otte (Company)





          2004-05-04 17:44:47 UTC
        



ok, it doesn't segfault anymore and errors out properly. I'll leave this open so
matroska gets fixed.






Comment 5


Arwed v. Merkatz





          2004-07-30 17:41:03 UTC
        



Created attachment 30089 [details] [review]
patch to make demuxing of vorbis in matroska  work

This patch works for me. I'm not sure if it's ok to check for the beginning of
the stream like I do in the patch, but it worked with the files I tested.