F-Spot crashes during authorization exporting to Facebook if friend IDs are large

Bug #405005 reported by Andreas Sandberg
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Critical
f-spot (Ubuntu)
Fix Released
Medium
Ubuntu Desktop Bugs

Bug Description

Binary package hint: f-spot

F-spot version 0.5.0.3-1ubuntu6 running on 64 bit Jaunty crashes when exporting to Facebook if a friend has a large ID. This was fixed by change set e8986b85c5257a3ef5deb60c5531296de32c5fd0 in the F-spot GIT repository.

The following exception is generated:

Marshaling clicked 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: There is an error in XML document. ---> System.InvalidOperationException: <friends_get_response xmlns='http://api.facebook.com/1.0/'> was not expected
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot (System.Xml.Serialization.XmlTypeMapping rootMap) [0x00000]
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot () [0x00000]
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00000]
  --- End of inner exception stack trace ---
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00000]
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader) [0x00000]
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.IO.Stream stream) [0x00000]
  at Mono.Facebook.Util.GetResponse[FriendsResponse] (System.String method_name, Mono.Facebook.FacebookParam[] parameters) [0x00000]
  at Mono.Facebook.FacebookSession.GetFriends () [0x00000]
  at FSpot.Exporter.Facebook.FacebookExport.HandleLoginClicked (System.Object sender, System.EventArgs args) [0x00000]
  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.Application.gtk_main()
   at Gtk.Application.Run()
   at Gnome.Program.Run()
   at FSpot.Driver.Main(System.String[] args)

Revision history for this message
Andreas Sandberg (andysan) wrote :
Revision history for this message
Andreas Sandberg (andysan) wrote :

Note that this is different from #363067 which (originally) dealt with an unexpected HTTP response code. Some of the comments on that bug are really related to this bug.

Revision history for this message
Pedro Villavicencio (pedro) wrote :

Thanks for the report Andreas, that's just a part of the commit full change is at http://git.gnome.org/cgit/f-spot/commit/?id=edb2707703b8faf4154a12875edff8644e993325 and upstream bug is http://bugzilla.gnome.org/show_bug.cgi?id=585902 ; Linking the report

Changed in f-spot (Ubuntu):
assignee: nobody → Ubuntu Desktop Bugs (desktop-bugs)
importance: Undecided → Medium
status: New → Fix Committed
Changed in f-spot:
status: Unknown → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.6.0.0-1~ubuntu1

---------------
f-spot (0.6.0.0-1~ubuntu1) karmic; urgency=low

  * Upload to Karmic from Debian's git VCS (LP: #410520)
  * New Upstream Version 0.6.0.0
   - no longer depends on libeog
   - fighting against deprecation, on the path to gnome3
   - new viewer widget
   - new widget showing folders organisation
   - new Preference dialog
   - db and config moved to .config (or whatever is pointed by xdg config)
   - tons of bugs fixed (really) (Closes: #534566, #449293, #479176, #305306,
     #537942, #418356)
     (LP: #77133, #290564, #298283, #379373, #405005, #116796, #178303,
     #355035, #368275)
   - updated translations
  * debian/control: Set Vcs-* to point to new Git locations
  * debian/control, debian/patches/*: Switch to quilt for patches, refresh all
    patches
  * debian/rules, debian/control: Rewrite in DH7 style (Closes: #531769)
  * debian/rules: Use ltmain-as-needed, modify LDFLAGS
  * debian/patches/99_ltmain_as-needed.dpatch: Drop, now applied in rules by
    copying from debian/
  * debian/patches/debian_unlink_nunit.dpatch: Drop, upstream now handles this
    case correctly by configure checks
  * debian/patches/debian_no_dup_app_icons.dpatch: Drop, upstream no longer
    ships svg icon
  * debian/patches/debian_link_2.0_profile_cairo.dpatch: Drop, fixed in mono
  * debian/patches/debian_dont_link_libmono.patch: Drop, ltmain-as-needed
    achieves this too
  * debian/patches/f-spot-import-gphoto.dpatch,
    debian/patches/svn-r4555_fix-crash-on-icon-size-change.dpatch,
    debian/patches/svn-r4545_locales-import.dpatch,
    debian/patches/debian_use_csc.dpatch,
    ubuntu_1024x600_CDExport.cs.dpatch,
    ubuntu_raw_escape_quote.dpatch (Closes: #481124),
    svn-display-sidebar.dpatch,
    svn-no-versionellipze.patch: Drop, fixed upstream
  * debian/control: Bump versions on a load of build-deps to match upstream
    configure checks. Also drop a load more build-deps that are no longer
    needed.
  * debian/{docs,menu,manpages}: Prefix with f-spot
  * debian/rules, debian/f-spot.install: Move DLLmap installation to
    f-spot.install instead of cp in rules
  * debian/gkeyfile-sharp.dll.config: Add, DLLImport was incorrect

 -- Iain Lane <email address hidden> Mon, 10 Aug 2009 17:42:29 +0100

Changed in f-spot (Ubuntu):
status: Fix Committed → Fix Released
Changed in f-spot:
importance: Unknown → Critical
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.