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 617650 - [mmssrc] should check location
[mmssrc] should check location
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 0.10.19
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-05-04 13:27 UTC by Julien Isorce
Modified: 2010-06-05 15:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Julien Isorce 2010-05-04 13:27:14 UTC 
* Steps to reproduce:

gst-launch mmssrc location=mms://a

or

gst-launch uridecodebin uri=mms:

* Results:

Program received signal SIGSEGV, Segmentation fault.

+ Trace 221709

Thread 3081488064 (LWP 1941)

  • #0 mms_connect
    from /usr/lib/libmms.so.0
  • #1 mmsx_connect
    from /usr/lib/libmms.so.0
  • #2 gst_mms_start
    at gstmms.c line 432
  • #3 gst_base_src_start 

* This is abviously a problem from libmms0.4-2 but gstmmssrc should prevent from this crash.
Comment 1 Sebastian Dröge (slomo) 2010-05-06 07:52:37 UTC 
commit 42ac5097a091f8cde65ed592ef6b1aba610ffddc
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Thu May 6 09:48:00 2010 +0200

    mmssrc: Check URI for validity and work around URI parsing bugs in libmms
    
    Fixes bug #617650.
Comment 2 Hans de Goede 2010-06-05 15:58:22 UTC 
Hi,

Although I appreciate you putting the workaround in place, I would like to point out that the (pretty recent) new upstream 0.6 release fixes this in libmms where the fix belongs, in combination with a whole slew of other bugfixes, see:

https://sourceforge.net/projects/libmms/files/libmms/0.6/rel-notes/view

As testing / bug reports have shown that this release fixes many streams which would previously not playback it might be an idea to drop this "fix" from mmssrc and instead just require >= 0.6 in the pkgconfig check in the next bad release.

Regards,

Hans (upstream libmms developer)