Recent Documents crash when cleared

Bug #580723 reported by Robert Dyer
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Docky
Fix Released
Low
Robert Dyer
docky (Ubuntu)
Fix Released
Undecided
Unassigned
Lucid
Fix Released
Undecided
Unassigned

Bug Description

Reporting this for another user.

Original report here: https://bugs.launchpad.net/docky/+bug/464358/comments/1

Revision history for this message
Robert Dyer (psybers) wrote :

I believe I fixed this in rev 1340. Reopen THIS bug if it did not fix it.

Changed in docky:
assignee: nobody → Robert Dyer (psybers)
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
Dag Odenhall (dag.odenhall) wrote :

Still present in r1343 PPA, want me to check stack trace again? Sorry about the previous misposting, was just trying to avoid duplicates but misinterpreted the meaning of "deleted".

Revision history for this message
Robert Dyer (psybers) wrote :

Yes please post your stack trace because I can't reproduce this bug.

Changed in docky:
status: Fix Committed → Incomplete
Changed in docky:
milestone: none → 2.0.4
Revision history for this message
Robert Dyer (psybers) wrote :

@Dag: can you please re-confirm this bug is still happening? And if so please post your new stack trace.

Changed in docky:
milestone: 2.0.4 → 2.0.5
Changed in docky:
milestone: 2.0.5 → none
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted docky into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Iain Lane (laney) wrote :

This functionality still seems to work in the -proposed package. But I didn't have the original crash. Seems no regression though.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Dag Odenhall (dag.odenhall) wrote :
Download full text (5.9 KiB)

Marshaling changed 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.InvalidOperationException: Operation is not valid due to the current state of the object
  at System.Linq.Enumerable.First[FileDockItem] (IEnumerable`1 source) [0x00000]
  at RecentDocuments.RecentDocumentsItem.UpdateInfo () [0x00051] in /build/buildd/docky-2.1.0~bzr1589/StandardPlugins/RecentDocuments/src/RecentDocumentsItem.cs:128
  at RecentDocuments.RecentDocumentsItem.RefreshRecentDocs () [0x0010f] in /build/buildd/docky-2.1.0~bzr1589/StandardPlugins/RecentDocuments/src/RecentDocumentsItem.cs:118
  at RecentDocuments.RecentDocumentsItem.<RecentDocumentsItem>m__2 (System.Object , System.EventArgs ) [0x00000] in /build/buildd/docky-2.1.0~bzr1589/StandardPlugins/RecentDocuments/src/RecentDocumentsItem.cs:95
  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]
  --- 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]
  at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000]
  at System.Delegate.DynamicInvokeImpl (System.Object[] args) [0x00000]
  at System.MulticastDelegate.DynamicInvokeImpl (System.Object[] args) [0x00000]
  at System.Delegate.DynamicInvoke (System.Object[] args) [0x00000]
  at GLib.Signal.ClosureInvokedCB (System.Object o, GLib.ClosureInvokedArgs args) [0x00000]
  at GLib.SignalClosure.Invoke (GLib.ClosureInvokedArgs args) [0x00000]
  at GLib.SignalClosure.MarshalCallback (IntPtr raw_closure, IntPtr return_val, UInt32 n_param_vals, IntPtr param_values, IntPtr invocation_hint, IntPtr marshal_data) [0x00000]
   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.RecentManager.gtk_recent_manager_purge_items(IntPtr , IntPtr ByRef )
   at Gtk.RecentManager.PurgeItems()
   at RecentDocuments.RecentDocumentsItemProvider+<ClearRecent>c__AnonStorey0.<>m__0(System.Object o, Gtk.ResponseArgs args) in /build/buildd/docky-2.1.0~bzr1589/StandardPlugins/RecentDocuments/src/RecentDocumentsItemProvider.cs:line 71
   at System.Reflection.MonoMethod.InternalInvoke(System.Object , System.Object[] , System.Exception ByRef )
   at System.Reflection.MonoMethod.Invoke(System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(System.Object obj, System....

Read more...

Revision history for this message
Robert Dyer (psybers) wrote :

Fixed in rev 1593.

Changed in docky:
status: Incomplete → Fix Committed
milestone: none → 2.0.6
Changed in docky:
status: Fix Committed → Fix Released
Revision history for this message
Rico Tzschichholz (ricotz) wrote :

debdiff which fixes this is attached to bug #584094

tags: removed: verification-done
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted docky into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in docky (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-2

---------------
docky (2.0.6-2) unstable; urgency=low

  * Upload to unstable.

docky (2.0.6-1) experimental; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
  * debian/patches:
    + drop 01_use_cli_for_wrapper.patch
  * patch build-system and bump dependency for new gio-sharp >= 2.22.2
    + update debian/patches/use-system-gio-sharp.patch
 -- Iain Lane <email address hidden> Tue, 07 Sep 2010 09:03:13 +0100

Changed in docky (Ubuntu):
status: New → Fix Released
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Verification for lucid

I've not been able to reproduce the original crash with docky 2.0.5-0ubuntu1 in lucid-updates, but I have verified that the version of docky 2.0.6-0ubuntu1 in -proposed doesn't crash either and is working correctly. Marking as verification-done

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package docky - 2.0.6-0ubuntu1

---------------
docky (2.0.6-0ubuntu1) lucid-proposed; urgency=low

  * New upstream release 2.0.6 "Just lovin' it",
    changes include:
    + fix check for existing instance
    + hitting ESC should cancel drags (LP: #460071)
    + make sure plugins enabled by default are added to the dock on first run
      and enable Trash docklet by default
    + update badge drawing so the text is consistently sized
    + fix leak of ThemedIcon/FileIcon in IconFromGIcon
    + fix non-square items arent properly darken (LP: #584094)
    + remove MenuList.Dispose which is not really needed, solves (LP: #609170)
    + shrink config dialog (LP: #591448)

    + RecentDocs: fix crash (LP: #580723)
    + Mounter: use a better MountItem.UniqueID and don't fill up gconf with
      useless keys (temporary fix for LP: #600311)
    + Weather: add weather icon for Smoke (LP: #612437)
    + CPUMonitor: fix CPU utilization percentage (LP: #604677)
    + GMail:
      - add debuging messages to gmail and log web response status when we
        have a web exception
      - fix crash and leak in Gmail
 -- Rico Tzschichholz <email address hidden> Wed, 01 Sep 2010 10:20:15 +0200

Changed in docky (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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