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 745072 - view-model: Crash caused by info-updated emission after main window is destroyed
view-model: Crash caused by info-updated emission after main window is destroyed
Status: RESOLVED FIXED
Product: gnome-photos
Classification: Applications
Component: general
3.14.x
Other All
: Normal normal
: ---
Assigned To: GNOME photos maintainer(s)
GNOME photos maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-02-24 09:23 UTC by Debarshi Ray
Modified: 2015-02-25 08:45 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
view-model: Fix the lifetime of the info-updated handler (1.41 KB, patch)
2015-02-24 09:24 UTC, Debarshi Ray 		committed Details | Review

Description Debarshi Ray 2015-02-24 09:23:33 UTC 
After fixing bug 745070 and bug 745071, if you close the main window soon after launching the application, you get a crash like this:

Program terminated with signal SIGSEGV, Segmentation fault.

+ Trace 234707

  • #0 g_type_check_instance_cast
    at gtype.c line 4060
  • #0 g_type_check_instance_cast
    at gtype.c line 4060
  • #1 photos_view_model_info_updated
    at photos-view-model.c line 245
  • #2 g_closure_invoke
    at gclosure.c line 768
  • #3 signal_emit_unlocked_R
    at gsignal.c line 3549
  • #4 g_signal_emit_valist
    at gsignal.c line 3305
  • #5 g_signal_emit
    at gsignal.c line 3361
  • #6 photos_base_item_check_effects_and_update_info
    at photos-base-item.c line 305
  • #7 photos_base_item_create_thumbnail_in_thread_func
    at photos-base-item.c line 335
  • #8 photos_base_item_refresh_collection_icon
    at photos-base-item.c line 496
  • #9 g_cclosure_marshal_VOID__OBJECTv
    at gmarshal.c line 2102
  • #10 _g_closure_invoke_va
    at gclosure.c line 831
  • #11 g_signal_emit_valist
  • #12 g_signal_emit
    at gsignal.c line 3361
  • #13 photos_collection_icon_watcher_create_collection_icon
    at photos-collection-icon-watcher.c line 99
  • #14 _g_closure_invoke_va
    at gclosure.c line 831
  • #15 g_signal_emit_valist
    at gsignal.c line 3214
  • #16 g_signal_emit
    at gsignal.c line 3361
  • #17 photos_base_item_check_effects_and_update_info
    at photos-base-item.c line 305
  • #18 photos_base_item_create_thumbnail_in_thread_func
    at photos-base-item.c line 335
  • #19 photos_base_item_refresh_thumb_path_pixbuf
    at photos-base-item.c line 567
  • #20 g_task_return_now
    at gtask.c line 1077
  • #21 complete_in_idle_cb
    at gtask.c line 1086
  • #22 g_main_dispatch
    at gmain.c line 3122
  • #23 g_main_context_dispatch
    at gmain.c line 3737
  • #24 g_main_context_iterate
    at gmain.c line 3808
  • #25 g_main_context_iteration
    at gmain.c line 3869
  • #26 g_application_run
    at gapplication.c line 2327
  • #27 gegl_gtk_view_autoscale_get_type
    at gegl-gtk-enums.c line 24
  • #28 __libc_start_main
    from /lib64/libc.so.6
  • #29 _start 






Comment 1


Debarshi Ray





          2015-02-24 09:24:48 UTC
        



Created attachment 297743 [details] [review]
view-model: Fix the lifetime of the info-updated handler