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 170329 - Aborting File Add can sometimes crash the application
Aborting File Add can sometimes crash the application
Status: RESOLVED FIXED
Product: file-roller
Classification: Applications
Component: general
2.6.x
Other All
: High critical
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks:
 
 
Reported: 2005-03-14 14:47 UTC by laszlo.kovacs
Modified: 2005-03-14 15:02 UTC
See Also:
GNOME target: ---
GNOME version: 2.5/2.6


Description laszlo.kovacs 2005-03-14 14:47:14 UTC 
Steps to reproduce:
1. Create new archive.
2. Add a very large file to it, large enough to have the Adding FIle to Archive
dialog displayed for a while.
3. Close the dialog by clicking on the relevant window manager decoration icon.
4. A crash can occur.


Stack trace:

+ Trace 56852

  • #0 g_type_check_instance_cast
    at gtype.c line 3159
  • #1 window_stop_activity_mode
    at window.c line 4504
  • #2 update_data_free
    at window.c line 636
  • #3 update_file_list_idle
    at window.c line 657
  • #4 window_update_file_list
    at window.c line 821
  • #5 _action_performed
    at window.c line 1684
  • #6 fr_marshal_VOID__INT_POINTER
    at fr-marshal.c line 84
  • #7 g_closure_invoke
    at gclosure.c line 437
  • #8 signal_emit_unlocked_R
    at gsignal.c line 2478
  • #9 g_signal_emit_valist
    at gsignal.c line 2237
  • #10 g_signal_emit
    at gsignal.c line 2281
  • #11 action_performed
    at fr-archive.c line 484
  • #12 fr_marshal_VOID__INT_POINTER
    at fr-marshal.c line 84
  • #13 g_closure_invoke
    at gclosure.c line 437
  • #14 signal_emit_unlocked_R
    at gsignal.c line 2478
  • #15 g_signal_emit_valist
    at gsignal.c line 2237
  • #16 g_signal_emit
    at gsignal.c line 2281
  • #17 fr_command_done
    at fr-command.c line 246
  • #18 g_cclosure_marshal_VOID__POINTER
    at gmarshal.c line 601
  • #19 g_closure_invoke
    at gclosure.c line 437
  • #20 signal_emit_unlocked_R
    at gsignal.c line 2478
  • #21 g_signal_emit_valist
    at gsignal.c line 2237
  • #22 g_signal_emit
    at gsignal.c line 2281
  • #23 check_child
    at fr-process.c line 725
  • #24 g_timeout_dispatch
    at gmain.c line 3301
  • #25 g_main_dispatch
    at gmain.c line 1942
  • #26 g_main_context_dispatch
    at gmain.c line 2492
  • #27 g_main_context_iterate
    at gmain.c line 2573
  • #28 g_main_loop_run
    at gmain.c line 2777
  • #29 gtk_main
    at gtkmain.c line 1173
  • #30 main
    at main.c line 125 


Other information:
Comment 1 laszlo.kovacs 2005-03-14 14:48:37 UTC 
The problem is that progress_dialog_delete_event() should be gboolean returning
TRUE to avoid calling the default signal handler as well which destroys the widget.

Don't have a patch as I worked on this on the 2.6 source tree, but it is very
easy to fix.
Comment 2 Paolo Bacchilega 2005-03-14 15:02:06 UTC 
fixed now in CVS HEAD, thank you.