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 693194 - crash obtaining avatar icon data (google account)
crash obtaining avatar icon data (google account)
Status: RESOLVED NOTGNOME
Product: gnome-contacts
Classification: Core
Component: general
3.6.x
Other Linux
: Normal major
: ---
Assigned To: GNOME Contacts maintainer(s)
GNOME Contacts maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-02-05 11:22 UTC by Christian Kirbach
Modified: 2013-06-08 22:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Christian Kirbach 2013-02-05 11:22:53 UTC 
Hi

I have only one online account configured in online-accounts: google

Usually g-c-search-provider crashes at session login. according to appport crash handler, there is Ubuntu bug # 1033262 about it, but it is hidden. I did not find any bugzilla bug with "1033262" in a comment, thus assuming it has not been reported upstream.

g_variant_new: expected GVariant of type `ay' but received value has type `(null)'


Core was generated by `/usr/lib/gnome-contacts/gnome-contacts-search-provider'.
Program terminated with signal 5, Trace/breakpoint trap.

+ Trace 231487

  • #0 g_logv
    at /build/buildd/glib2.0-2.34.1/./glib/gmessages.c line 974
  • #0 g_logv
    at /build/buildd/glib2.0-2.34.1/./glib/gmessages.c line 974
  • #1 g_log
    at /build/buildd/glib2.0-2.34.1/./glib/gmessages.c line 1003
  • #2 g_variant_valist_new_nnp
    at /build/buildd/glib2.0-2.34.1/./glib/gvariant.c line 4155
  • #3 g_variant_valist_new_leaf
    at /build/buildd/glib2.0-2.34.1/./glib/gvariant.c line 4311
  • #4 g_variant_valist_new
    at /build/buildd/glib2.0-2.34.1/./glib/gvariant.c line 4493
  • #5 g_variant_valist_new
    at /build/buildd/glib2.0-2.34.1/./glib/gvariant.c line 4545
  • #6 g_variant_new_va
  • #7 g_variant_new
  • #8 contacts_contact_get_avatar_icon_data
  • #9 contacts_search_provider_GetResultMetas
  • #10 ??
  • #11 ??
  • #12 call_in_idle_cb
    at /build/buildd/glib2.0-2.34.1/./gio/gdbusconnection.c line 4737
  • #13 g_main_dispatch
    at /build/buildd/glib2.0-2.34.1/./glib/gmain.c line 2715
  • #14 g_main_context_dispatch
    at /build/buildd/glib2.0-2.34.1/./glib/gmain.c line 3219
  • #15 g_main_context_iterate
    at /build/buildd/glib2.0-2.34.1/./glib/gmain.c line 3290
  • #16 g_main_context_iteration
    at /build/buildd/glib2.0-2.34.1/./glib/gmain.c line 3351
  • #17 g_application_run
    at /build/buildd/glib2.0-2.34.1/./gio/gapplication.c line 1624
  • #18 _vala_main
  • #19 __libc_start_main
    at libc-start.c line 226
  • #20 _start 






Comment 1


Alexander Larsson





          2013-02-06 09:18:07 UTC
        



Thats a weird crash, its like the pixel_data is NULL in Contact.avatar_get_data, but i don't see how that could happen...






Comment 2


darkxst





          2013-04-04 05:15:11 UTC
        



I can reproduce this quite easily.

Start with a search that brings up a list of contacts results, once that is displayed then add a couple more letters to this same search so now there are no matches.

This crashes the search provider and you are left with an empty contacts pane in the search overview.






Comment 3


Alfonso





          2013-04-19 12:30:29 UTC
        



This happens to me too with ubuntu, but not with arch linux (think it didn't happen with fedora either, but can't say 100% sure). Ubuntu has two different online-accounts entries. Appart from the gnome-online-accounts they have their own one. I wonder if this could be related to this bug...

https://bugs.launchpad.net/ubuntu/+source/gnome-control-center/+bug/1040193






Comment 4


darkxst





          2013-04-20 02:44:30 UTC
        



I can't imagine it would be related to Ubuntu-online-accounts, since gnome-contacts has not been patched to support that.

However Ubuntu do patch the following line, to be an ARRAY instead of BYTESTRING. 

var pixel_data = Variant.new_from_data (VariantType.BYTESTRING,
                                        small_avatar.get_pixels_with_length (),
                                        true, small_avatar);

https://git.gnome.org/browse/gnome-contacts/tree/src/contacts-contact.vala#n168






Comment 5


darkxst





          2013-04-20 03:02:07 UTC
        



I cannot reproduce the crash with the above patch removed. So this is a downstream bug.






Comment 6


Jeremy Bicha





          2013-06-08 22:36:15 UTC
        



Thanks Tim for figuring this out.