gnome-mplayer crashed with SIGSEGV in g_ascii_strncasecmp()

[Copernicvs]

Binary package hint: gnome-mplayer

jaunty 2.6.28.11
gnome mplayer 0.9.4
i added a few gigabytes of mp3 files to the playlist, and gmplayer crashed while loading them

ProblemType: Crash
Architecture: i386
DistroRelease: Ubuntu 9.04
ExecutablePath: /usr/bin/gnome-mplayer
NonfreeKernelModules: nvidia
Package: gnome-mplayer 0.9.4-1
ProcCmdline: gnome-mplayer
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_HK.UTF-8
 SHELL=/bin/bash
Signal: 11
SourcePackage: gnome-mplayer
StacktraceTop:
 g_ascii_strncasecmp () from /usr/lib/libglib-2.0.so.0
 ?? ()
 ?? ()
 ?? ()
 ?? ()
Title: gnome-mplayer crashed with SIGSEGV in g_ascii_strncasecmp()
Uname: Linux 2.6.28-11-generic i686
UserGroups: adm admin audio cdrom dialout dip floppy fuse lpadmin netdev plugdev video

[Apport retracing service (apport)]

StacktraceTop:IA__g_ascii_strncasecmp (
device_name (
parse_basic (
parse_playlist (
add_item_to_playlist_callback (data=0x95ad408,

[Cesare Tirabassi (norsetto)]

Could you include the listing of the directory (and its subdirectories) which gives the fault? It should be the one named "Tommy Emmanuel Albums". I suspect it could be an ill-formed name that causes the crash.

[Copernicvs (kornenator)]

it would be quite hard to include all the directories. i do have a
"Tommy Emmanuel Albums" too, but if i add it separately, then everything
is fine. if it's an ill-formatted directory (structure), then i'd say it
should rather be one of the chinese albums i have. but actually, i'd
suspect some error in id3 tags, that causes the crash. and btw, it
effects every playr i have tried so far, except exaile (but exaile
crashes randomly anyway...). audacious, rhythmbox, gnome-mplayer,
listen...

On Sun, 2009-04-05 at 22:23 +0000, Cesare Tirabassi wrote:

> Could you include the listing of the directory (and its subdirectories)
> which gives the fault? It should be the one named "Tommy Emmanuel
> Albums". I suspect it could be an ill-formed name that causes the crash.
>
> ** Changed in: gnome-mplayer (Ubuntu)
> Status: New => Incomplete
>

[Cesare Tirabassi (norsetto)]

OK, thanks for letting us know. If you do happen to identify the file that is causing the crash it could be interesting if we could have a look at it. This could help us to pin down the particular cause for the crash.

[Copernicvs (kornenator)]

i'd be glad to help, but do ou think there's a way to create a log of
the files it opens? then the last one should be the "killer".
sorry for such questions, but i'm not an expert on linux systems, barely
started using Ubuntu a while ago.

On Mon, 2009-04-06 at 09:52 +0000, Cesare Tirabassi wrote:

> OK, thanks for letting us know. If you do happen to identify the file
> that is causing the crash it could be interesting if we could have a
> look at it. This could help us to pin down the particular cause for the
> crash.
>

[Cesare Tirabassi (norsetto)]

Try launching the application from the command line, like this:

gnome-mplayer --verbose > error.log

After the crash there should be a file named error.log in your home directory which could contain some info about the item that caused the crash. Check it out and eventually attach it to this bug report.

[Copernicvs (kornenator)]

here's the error file, i attached it.
the last file in the list is a WMA, which was originally on my Meizu M6
mp3 player as a test song. it should be a version 8 lossless WMA, but if
i open that one just by itself, then gnome-mplayer doesn't have any
problems.

On Mon, 2009-04-06 at 20:22 +0000, Cesare Tirabassi wrote:

> Try launching the application from the command line, like this:
>
> gnome-mplayer --verbose > error.log
>
> After the crash there should be a file named error.log in your home
> directory which could contain some info about the item that caused the
> crash. Check it out and eventually attach it to this bug report.
>

[Cesare Tirabassi (norsetto)]

No, the faulty entry is the one after that. Apparently the callback is called with a blank entry, which causes the crash in a string comparison. I think upstream should add a length check before calling g_slist_foreach in menuitem_open_callback (its already checking that the pointer is not null but apparently its not enough), even though this doesn't explain how this entry gets added in the first place. I'll have to ask upstream to have a look at this.
In the mean time if you keep trying to found out what is the entry which is causing this it could help.

[Kevin DeKorte (kdekorte-gmail)]

I believe this patch should fix it.

Index: src/support.c
===================================================================
--- src/support.c (revision 1242)
+++ src/support.c (working copy)
@@ -1529,6 +1529,9 @@
     gchar *unescaped = NULL;
     MetaData *data = NULL;

+ if (strlen(uri) < 1)
+ return FALSE;
+
     if (verbose)
         printf("adding %s to playlist\n", uri);
     local_uri = strdup(uri);

please let me know if it does.

[Cesare Tirabassi (norsetto)]

The updated package (0.9.5-1~jaunty2) is available from here:

https://launchpad.net/~norsetto/+archive/ppa

It would be appreciated if you can test it and report here if this solves your problem.

[Copernicvs (kornenator)]

yesterday i updated to the 64bit RC, and also migrated all my stuff to
ext4 partitions. maybe i already got the update as via the automated
system, or it just doesn't effect the new system.
but anyhow, adding all my mp3 doesn't cause a crash anymore!

thanks for your help!

On Sat, 2009-04-18 at 15:48 +0000, Cesare Tirabassi wrote:
> The updated package (0.9.5-1~jaunty2) is available from here:
>
> https://launchpad.net/~norsetto/+archive/ppa
>
> It would be appreciated if you can test it and report here if this
> solves your problem.
>
> ** Changed in: gnome-mplayer (Ubuntu)
> Status: Triaged => Fix Committed
>

[Launchpad Janitor (janitor)]

This bug was fixed in the package gnome-mplayer - 0.9.6-1

---------------
gnome-mplayer (0.9.6-1) unstable; urgency=low

  * New upstream release:
    - new media tracker and audio meter
    - new key "use_pulse_flat_volume" to be set if you are using
      Pulse Audio 0.9.15 (see /usr/share/doc/gnome-mplayer/NEWS.Debian.gz
      and /usr/share/doc/gnome-mplayer/changelog.gz for additional info)
    - workaround for opening files on smb shares (LP: #369426)
    - prevent adding an "empty" filename to the playlist (LP: #348864)
    - add new --large_buttons and --always_hide_after_timeout command
      line options to man page
    - update debian/copyright with the newly added upstream source files
    - upstream is now widely using libtool, which is causing rpath and
      dependancies issues. libtool is really not needed for such a simple
      case so it has been patched out in favour of static linking:
      + updated debian/patches/avoid_libtool.patch to patch configure.in
        and Makefile.am instead of directly Makefile.in
      + debian/rules:
        - patch is now applied in the configure target
        - add aclocal/autoconf/automake calls in the configure target
      + debian/control:
        add autoconf and automake as Build-Depends
    - add debian/NEWS
  * Update debian/copyright in line with updated proposal specification
  * Add patch description to avoid_libtool.patch (thanks lintian)
  * Remove deprecated dh_desktop call
  * Use new dh_quilt_patch and dh_quilt_unpatch commands
  * No change bump of Standards-Version to 3.8.1
  * Changed Section to video (and debug for the dbg binary), in line
    with the new sections recently added to the Debian archive
  * Use LDFLAGS=-Wl,--as-needed instead of specifying manually all libraries

 -- Ubuntu Archive Auto-Sync <email address hidden> Mon, 08 Jun 2009 10:59:47 +0100