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 631277 - Regression: Crash when editing tags on unknown CD tracks
Regression: Crash when editing tags on unknown CD tracks
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: Importing
1.8.0
Other Linux
: Normal normal
: 1.x
Assigned To: Andrés G. Aragoneses (IRC: knocte)
Banshee Maintainers
: 631973 633564 634145 637172 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-10-04 04:16 UTC by James Mason
Modified: 2010-12-14 14:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch v1 (1.10 KB, patch)
2010-10-05 01:59 UTC, Andrés G. Aragoneses (IRC: knocte)
committed Details | Review

Description James Mason 2010-10-04 04:16:59 UTC
Tag attributes cannot be edited on CD tracks (such as when importing a CD for which metadata was not fetched) - Banshee crashes.

Steps to reproduce:

(1) Insert a CD for which metadata will not be fetched.
(2) Select a track from the 'Audio CD' list, and Edit Track Information...
(3) Change some attribute, and press 'Save'
(4) Watch the Banshee window disappear ;)

cmd line output:
Comment 1 James Mason 2010-10-04 04:18:26 UTC
cmd line output:


[Info  21:16:19.459] Querying MusicBrainz for Disc Release (qrIUicVH7WOxXlKoCp0lLDxdBVE-)
[Info  21:16:19.696] Query finished (success: False, 0.236435 seconds)
Marshaling response signal
Exception in Gtk# callback delegate
  Note: Applications can use GLib.ExceptionManager.UnhandledException to handle the exception.
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.NullReferenceException: Object reference not set to an instance of an object
  at Banshee.Collection.Database.DatabaseTrackInfo.Update () [0x00000] in <filename unknown>:0 
  at Banshee.Gui.TrackEditor.TrackEditorDialog.SaveTrack (Banshee.Gui.TrackEditor.EditorTrackInfo track) [0x00000] in <filename unknown>:0 
  at Banshee.Gui.TrackEditor.TrackEditorDialog.Save () [0x00000] in <filename unknown>:0 
  at Banshee.Gui.TrackEditor.TrackEditorDialog+<Run>c__AnonStorey11.<>m__78 (System.Object o, Gtk.ResponseArgs args) [0x00000] in <filename unknown>:0 
  at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (object,object[],System.Exception&)
  at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in <filename unknown>:0 
  --- End of inner exception stack trace ---
  at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in <filename unknown>:0 
  at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in <filename unknown>:0 
  at System.Delegate.DynamicInvokeImpl (System.Object[] args) [0x00000] in <filename unknown>:0 
  at System.MulticastDelegate.DynamicInvokeImpl (System.Object[] args) [0x00000] in <filename unknown>:0 
  at System.Delegate.DynamicInvoke (System.Object[] args) [0x00000] in <filename unknown>:0 
  at GLib.Signal.ClosureInvokedCB (System.Object o, GLib.ClosureInvokedArgs args) [0x00000] in <filename unknown>:0 
  at GLib.SignalClosure.Invoke (GLib.ClosureInvokedArgs args) [0x00000] in <filename unknown>:0 
  at GLib.SignalClosure.MarshalCallback (IntPtr raw_closure, IntPtr return_val, UInt32 n_param_vals, IntPtr param_values, IntPtr invocation_hint, IntPtr marshal_data) [0x00000] in <filename unknown>:0 
   at GLib.ExceptionManager.RaiseUnhandledException(System.Exception e, Boolean is_terminal)
   at GLib.SignalClosure.MarshalCallback(IntPtr raw_closure, IntPtr return_val, UInt32 n_param_vals, IntPtr param_values, IntPtr invocation_hint, IntPtr marshal_data)
   at Gtk.Application.gtk_main()
   at Gtk.Application.Run()
   at Banshee.Gui.GtkBaseClient.Run()
   at Banshee.Gui.GtkBaseClient.Startup()
   at Hyena.Gui.CleanRoomStartup.Startup(Hyena.Gui.StartupInvocationHandler startup)
   at Banshee.Gui.GtkBaseClient.Startup()
   at Banshee.Gui.GtkBaseClient.Startup(System.String[] args)
   at Nereid.Client.Main(System.String[] args)
   at System.AppDomain.ExecuteAssembly(System.Reflection.Assembly , System.String[] )
   at System.AppDomain.ExecuteAssemblyInternal(System.Reflection.Assembly a, System.String[] args)
   at System.AppDomain.ExecuteAssembly(System.String assemblyFile, System.Security.Policy.Evidence assemblySecurity, System.String[] args)
   at System.AppDomain.ExecuteAssembly(System.String assemblyFile)
   at Booter.Booter.BootClient(System.String clientName)
   at Booter.Booter.Main()
Comment 2 Adam Reeve 2010-10-04 06:36:52 UTC
I'm having the same issue and get the same backtrace. This also occurs when editing CD metadata that has been fetched.

I'm on 64 bit Ubuntu 10.04 using Banshee 1.8.0 from the Ubuntu PPA. Mono JIT compiler version 2.4.4 (Debian 2.4.4~svn151842-1ubuntu4).
Comment 3 Andrés G. Aragoneses (IRC: knocte) 2010-10-04 13:52:47 UTC
Ooops, my fault.
Comment 4 Andrés G. Aragoneses (IRC: knocte) 2010-10-05 01:59:38 UTC
Created attachment 171731 [details] [review]
Patch v1

Proposed fix.
Comment 5 Andrés G. Aragoneses (IRC: knocte) 2010-10-05 23:15:35 UTC
Comment on attachment 171731 [details] [review]
Patch v1

Thanks, I'll commit to the stable branch too.
Comment 6 Andrés G. Aragoneses (IRC: knocte) 2010-10-07 06:50:45 UTC
Was trying to push to the stable branch but found out that Gabriel already did :)

This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.
Comment 7 Michael Martin-Smucker 2010-10-12 13:28:15 UTC
*** Bug 631973 has been marked as a duplicate of this bug. ***
Comment 8 Bertrand Lorentz 2010-11-01 18:24:03 UTC
*** Bug 633564 has been marked as a duplicate of this bug. ***
Comment 9 Michael Martin-Smucker 2010-11-09 00:47:30 UTC
*** Bug 634145 has been marked as a duplicate of this bug. ***
Comment 10 Michael Martin-Smucker 2010-12-14 14:02:05 UTC
*** Bug 637172 has been marked as a duplicate of this bug. ***