muine crashes on adding music to the playlist

Bug #117726 reported by Joachim Beckers on 2007-05-30
54
This bug affects 1 person
Affects Status Importance Assigned to Milestone
muine (Ubuntu)
High
MOTU Mono Team
Nominated for Gutsy by Stephan Rügamer
Nominated for Hardy by Stephan Rügamer
Nominated for Intrepid by Stephan Rügamer

Bug Description

Binary package hint: muine

to reproduce this bug: run muine and click 'add song' or 'add album'. This bug can be reproduced 100% every time in gutsy and + hardy.

joachim@madkeunnen:~$ muine

***MEMORY-WARNING***: /usr/lib/muine/muine.exe[17436]: GSlice: g_thread_init() must be called before all other GLib functions; memory corruption due to late invocation of g_thread_init() has been detected; this program is likely to crash, leak or unexpectedly abort soon...

(muine:17436): Gtk-CRITICAL **: gtk_widget_grab_default: assertion `GTK_WIDGET_CAN_DEFAULT (widget)' failed
Error grabbing key 144, 0x8571800
Error grabbing key 153, 0x8571800
Error grabbing key 162, 0x8571800
Error grabbing key 162, 0x8571800
Error grabbing key 164, 0x8571800

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

(muine:17436): Gtk-CRITICAL **: gtk_tree_model_row_inserted: assertion `path != NULL' failed

[repeated a couple of hundred times]

Unhandled Exception: System.NullReferenceException: A null value was found where an object instance was required.
at Muine.AddSongWindow.GetTitle (Muine.Song) <0x0000d>
at Muine.AddSongWindow.SetText (Gtk.CellRendererText,Gtk.TreeIter) <0x0004d>
at Muine.AddSongWindow.CellDataFunc (Gtk.TreeViewColumn,Gtk.CellRenderer,Gtk.TreeModel,Gtk.TreeIter) <0x0004b>
at (wrapper delegate-invoke) System.MulticastDelegate.invoke_void_TreeViewColumn_CellRenderer_TreeModel_TreeIter (Gtk.TreeViewColumn,Gtk.CellRenderer,Gtk.TreeModel,Gtk.TreeIter) <0x00058>
at GtkSharp.TreeCellDataFuncWrapper.NativeCallback (intptr,intptr,intptr,Gtk.TreeIter&,intptr) <0x00133>
at (wrapper native-to-managed) GtkSharp.TreeCellDataFuncWrapper.NativeCallback (intptr,intptr,intptr,Gtk.TreeIter&,intptr) <0x00053>
in (unmanaged) 0xb6e77e70
at (wrapper managed-to-native) Gtk.TreeView.gtk_tree_view_set_cursor (intptr,intptr,intptr,bool) <0x00004>
at Gtk.TreeView.SetCursor (Gtk.TreePath,Gtk.TreeViewColumn,bool) <0x000aa>
at Muine.HandleView.SelectFirst () <0x0003d>
at Muine.AddWindow.Search () <0x00567>
at Muine.AddWindow.Reset () <0x00017>
at Muine.AddWindow.ResetFunc () <0x0000a>
at (wrapper delegate-invoke) System.MulticastDelegate.invoke_bool () <0x0002e>
at IdleProxy.Handler () <0x0002a>
at (wrapper native-to-managed) IdleProxy.Handler () <0x00047>
in (unmanaged) 0xb7e953b0
at (wrapper managed-to-native) Gtk.Application.gtk_main () <0x00004>
at Gtk.Application.Run () <0x00007>
at Muine.Global.Main (string[]) <0x005d3>

joachim@madkeunnen:~$

Related branches

Joachim Beckers (jbeckers) wrote :

note:

the first error message seems to be another issue, also reported in bugs #116870 and 116687

Joachim Beckers (jbeckers) wrote :

also note:

the second buch of errors are not an issue. last-exit displays them too, but works fine.

also also note:

I've already tried to remove the muine song database, cover database, config, ... but without success

Joachim Beckers (jbeckers) wrote :

see this thread on the muine mailing list for a description of the problem and a fix:
http://mail.gnome.org/archives/muine-list/2007-June/msg00000.html

I guess we need a patch to rip out the gsequence that muine ships and to fix the makefiles associated.

Changed in muine:
status: Unconfirmed → Confirmed
Joachim Beckers (jbeckers) wrote :

muine svn has a hack that fixes this

Luis Villa (luis-villa) wrote :

Still there in latest beta, FWIW. Clearly critical for muine- might as well not ship the package at all in the current state.

Same for me - I normally use Muine as my music player, but with Ubuntu Gutsy I am now hosed.

Pedro Villavicencio (pedro) wrote :

Marking it as fix committed since there's a fix in the upstream svn.

Changed in muine:
assignee: nobody → mono
importance: Undecided → Medium
status: Confirmed → Triaged
importance: Medium → High
Brian Burger (bburger) wrote :

Is this a duplicate of bug #133990?

Also, whatever fix is out there didn't seem to have made it into Gutsy Final. Should this bug report be rolled back from "Fix Committed" status?

michel (michel-alexander) wrote :

the bug is fixed in muine-0.8.8 which came out on october 20th. it's not yet in ubuntu's repositories and i didn't manage to compile the source from muine-player.org

Peter Adolphs (futzilogik) wrote :

I get the following (slightly different) error messages with muine 0.8.7-1.1ubuntu2 on gutsy when pressing the "Play Album" button:

---------------------------------------------------------------------------------------------------------
[several Gtk-CRITICAL messages removed]

(muine:7601): libmuine-CRITICAL **: g_sequence_ptr_get_data: assertion `!ptr->is_end' failed
Exception in Gtk# callback delegate
  Note: Applications can use GLib.ExceptionManager.UnhandledException to handle the exception.
System.NullReferenceException: A null value was found where an object instance was required.
  at Muine.AddAlbumWindow.GetCoverImage (Muine.Album album) [0x00000]
  at Muine.AddAlbumWindow.SetCoverImage (Gtk.CellRendererPixbuf cell, TreeIter iter) [0x00000]
  at Muine.AddAlbumWindow.PixbufCellDataFunc (Gtk.TreeViewColumn col, Gtk.CellRenderer cell, TreeModel model, TreeIter iter) [0x00000]
  at (wrapper delegate-invoke) System.MulticastDelegate:invoke_void_TreeViewColumn_CellRenderer_TreeModel_TreeIter (Gtk.TreeViewColumn,Gtk.CellRenderer,Gtk.TreeModel,Gtk.TreeIter)
  at GtkSharp.TreeCellDataFuncWrapper.NativeCallback (IntPtr tree_column, IntPtr cell, IntPtr tree_model, IntPtr iter, IntPtr data) [0x00000]

   at GLib.ExceptionManager.RaiseUnhandledException ()
   at GtkSharp.TreeCellDataFuncWrapper.NativeCallback ()
   at GtkSharp.TreeCellDataFuncWrapper.NativeCallback ()
   at Gtk.TreeView.gtk_tree_view_set_cursor ()
   at Gtk.TreeView.gtk_tree_view_set_cursor ()
   at Gtk.TreeView.SetCursor ()
   at Muine.HandleView.SelectFirst ()
   at Muine.AddWindow.Search ()
   at Muine.AddWindow.Reset ()
   at Muine.AddWindow.ResetFunc ()
   at System.MulticastDelegate.invoke_bool ()
   at GLib.Idle+IdleProxy.Handler ()
   at GLib.Idle+IdleProxy.Handler ()
   at Gtk.Application.gtk_main ()
   at Gtk.Application.gtk_main ()
   at Gtk.Application.Run ()
   at Muine.Global.Main ()
---------------------------------------------------------------------------------------------------------

The bug always appears when you press "Play Song" or "Play Album", making muine unusable in gutsy.

The bug is gone with a self-compiled muine-0.8.8.

rihasant (riku.hs) wrote :

I have the described problem in Gutsy, but what is kind of weird is that the same package of muine (0.8.7-1.1ubuntu2) appears to be working in Hardy.

Alexander Koenig (alex-lisas) wrote :

A rebuild of the package using the new upstream tarball (0.8.8) and dropping all patches except '20_desrt_trayicon_behaviour.patch' will result in a muine package that works flawlessly on gutsy.

Wouter Bolsterlee (uws) wrote :

Is there any chance of a new version of Muine uploaded to Ubuntu? This bug annoys a lot of people.

Wouter Bolsterlee (uws) wrote :

I've built Muine 0.8.8 packages since it apparently takes quite some time for Ubuntu to update to the latest release. You can find them here: http://uwstopia.nl/blog/2008/01/muine-0-8-8-for-ubuntu

Brian Burger (bburger) wrote :

Wouter, Ubuntu only updates packages on version release - so next round won't be until Ubuntu 8.04's release in April.

Your 0.8.8 debs work in 7.10 for me - thanks! I won't have to keep trying to like Rhythmbox now, hopefully!

Muine now seems to use a whole lot more resources than I remember from previous (official Ubuntu) releases, though - between 9-14% of CPU constantly while it's playing.

By contrast, Rhythmbox seems to be using about 2-6% while playing.

Is this an 0.8.8 issue, or did previous versions of Muine use this much CPU too, and I've just forgotten?

Wouter Bolsterlee (uws) wrote :

2008-01-18 klockan 03:06 skrev Brian Burger:
> Wouter, Ubuntu only updates packages on version release - so next round
> won't be until Ubuntu 8.04's release in April.

But this is not really about an update, it is about making a package
actually WORK again. Ubuntu could as well have applied just the patch that
fixes this issue (see upstream bug for the patches).

> Your 0.8.8 debs work in 7.10 for me - thanks! I won't have to keep
> trying to like Rhythmbox now, hopefully!

Thanks, you're welcome.

> Muine now seems to use a whole lot more resources than I remember from
> previous (official Ubuntu) releases, though
> [snip]
> Is this an 0.8.8 issue, or did previous versions of Muine use this much
> CPU too, and I've just forgotten?

I don't think Muine has changed that much between 0.8.7 and 0.8.8. However,
Muine uses Mono, GTK+ and GStreamer, so there might have been some changes
in those areas.

  mvrgr, Wouter

--
:wq mail <email address hidden>
                                                      web http://uwstopia.nl

i'm unclean :: a libertine -- placebo

Peter Adolphs (futzilogik) wrote :

See also https://wiki.ubuntu.com/StableReleaseUpdates: "NOTE: With Universe and Multiverse SRU's FTBFS, not installable, and segfault on startup ( e.g. completely un-usable ) can also be considered SRU Candidates"

muine is completely unusable in gutsy since it segfaults when you actually want to play something.

Wouter Bolsterlee (uws) wrote :

I'm really sorry to say this but the Ubuntu update policy for issues like these is just PLAIN STUPID. How can you REFUSE to fix a CRITICAL BUG because of some stupid policies? Meanwhile upstream Muine developers (me included) get the crap from Ubuntu users suffering from this problem.

My packages at http://uwstopia.nl/blog/2008/01/muine-0-8-8-for-ubuntu for i386 and (contributed by someone else) powerpc work fine, so users may want to use those while Ubuntu ignores the complaints from its users.

Changed in muine:
status: Triaged → Confirmed

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Wouter,

Thanks for all of your work and attention on this, I understand your
frustration about this - I've frequently been a victim of Ubuntus broken
Muine packages my self.

Wouter Bolsterlee (uws) wrote:
> I'm really sorry to say this but the Ubuntu update policy for issues
> like these is just PLAIN STUPID. How can you REFUSE to fix a CRITICAL
> BUG because of some stupid policies? Meanwhile upstream Muine developers
> (me included) get the crap from Ubuntu users suffering from this
> problem.

No-one has refused anything, it's just that no-one has put the work into
 getting the fix into Ubuntu.

The policy linked above has the full description but basically someone
needs to:
 * Grab the upstream patch
 * Make a debdiff.
 * Get the upload sponsored
 * Get some people to confirm the upload fixed the problem
(along with some editing of this bug report)

I'm sorry that this hasn't happened yet but Muine is usually a volunteer
maintained package, and no-one has volunteered to do this stuff.

Dean
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHp6jdeedO8dcp9nYRAq0zAJ95j85LgSy58vRZLhVlC/JgHCfQNACePbi5
wm66MR6fCBXoCz6yMbjbz/Q=
=XURE
-----END PGP SIGNATURE-----

Dean Sas (dsas) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dean Sas wrote:
> I'm sorry that this hasn't happened yet but Muine is usually a volunteer
> maintained package, and no-one has volunteered to do this stuff.

Or indeed, I'm sorry to say, updated the version in Hardy yet.

Dean
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHp6qSeedO8dcp9nYRAu8eAKCUn4aT6yKa/bb5BM0imycryspAmgCfawvc
DXNaqxN9Y6tWTr2Fj4MxDsQ=
=ab7x
-----END PGP SIGNATURE-----

I also get this problem. What a shame! This should most definitely be marked as a bug to be patched via auto-updates. Otherwise it should simply be removed from the repos as it just doesn't work.

I seem to get the same error in hardy!

Dean Sas (dsas) on 2008-03-07
description: updated
Wouter Bolsterlee (uws) wrote :

Try my packages or the recently uploaded Muine 0.8.8 debs for Debian: http://packages.debian.org/muine

Wouter Bolsterlee (uws) wrote :

And another complaint filed upstream: http://bugzilla.gnome.org/show_bug.cgi?id=526225

UBUNTU: PLEASE DON'T ANNOY UPSTREAM SO MUCH AS YOU CURRENTLY DO. PLEASE GET YOUR ACT TOGETHER AND STOP BEING LAME FOR SO LONG.

Love,

Wouter Bolsterlee

Wouter Bolsterlee (uws) wrote :

Now for some constructive comments. The issue is the GSequence addition to GLib, which clashes wit hthe GSequence code in old Muine releases. These upstream commits are relevant to this issue afaics:

http://svn.gnome.org/viewvc/muine?view=revision&revision=1143

2007-06-05 Wouter Bolsterlee <email address hidden>

 * libmuine/gsequence.c: (g_old_sequence_new),
 * libmuine/gsequence.h:
 * libmuine/pointer-list-model.c: (pointer_list_model_get_iter),
 * libmuine/pointer-list-model.h:

 Renamed all occurences of GSequence* to GOldSequence*
 and all occurences of g_sequence_* to g_old_sequence_*.
 This is a very ugly work-around to fix the build with
 newer GLib > 2.12 which has an incompatible GSequence
 API.

http://svn.gnome.org/viewvc/muine?view=revision&revision=1149

2007-08-08 Wouter Bolsterlee <email address hidden>

 Update to newer GSequence API. Partly fixes bug #464825.
 Initial patch by Scott Robinson, but slightly changed
 because we cannot depend on Glib > 2.13 as of yet (no
 stable release!).

 * libmuine/gsequence.c:
 * libmuine/gsequence.h:

 Copied from Glib trunk with minor modifications to make
 it build: removed galias references and changed glib.h
 include statement.

 * libmuine/pointer-list-model.c:
 * libmuine/pointer-list-model.h:

 Change to Glib GSequence API.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package muine - 0.8.8-1ubuntu1

---------------
muine (0.8.8-1ubuntu1) intrepid; urgency=low

  * Merge from debian unstable, remaining changes:
    - Add recommends for gnome-icon-theme for KDE users
  * debian/patches/11_playlist_window.diff:
    - Added patch to fix FTBFS because of wrong named variable.
      Patch provided by Jo Shields <email address hidden>
      Thanks Jo.
  * Closes (LP: #117726) (LP: #122733)

muine (0.8.8-1) unstable; urgency=low

  * QA upload.
    + Set maintainer to Debian QA Group <email address hidden>.
  * Acknowledge NMUs.
  * Fix watch file. (Closes: #449835).
  * New upstream release.
    + Drop all previous patches.
    + Updated NDesk managed DBus import. (Closes: #440817, #427263).
    + Use external NDesk dbus. (Closes: #415419).
  * Move build system from CDBS to debhelper.
  * Move from simple-patchsys to quilt.
  * Remove -X revisions on build-depends.
  * Add authors and copyright holders in debian/copyright.
    + Point link to GPL-2.
  * Remove unneeded dirs file.
  * Escape hyphens in manpage.
  * 10_desktop_encoding.diff. (Remove deprecated encoding from desktop file.)
  * Bump Standards Version to 3.7.3.
    + Menu policy transition.

 -- Stephan Hermann <email address hidden> Thu, 15 May 2008 11:45:45 +0200

Changed in muine:
status: Confirmed → Fix Released
Keywan Tonekaboni (prometoys) wrote :

Hi,

I know you MOTU do a great work. But I was really sad to see, that muine in hardy show still the same behavior. Could someone find time and love to integrate the patch/package from uws in ubuntu? And please not in 8.10. Muine is since Gutsy totally unusable. There are a few people outside, who dont like banshee, rhythmbox etc and please allow them to use muine in ubuntu...

Regards,

Keywan

I confirm that the muine (0.8.7) is broken under Ubuntu 8.04, but the <http://uwstopia.nl/blog/2008/01/muine-0-8-8-for-ubuntu> packages work fine.

Keywan Tonekaboni (prometoys) wrote :

On Ubuntu 8.10 muine works now fine....

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.