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 491210 - Closing find dialog while composing reply closes Evolution
Closing find dialog while composing reply closes Evolution
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.12.x (obsolete)
Other All
: High critical
: ---
Assigned To: Bharath Acharya
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2007-10-28 22:27 UTC by Yuri Pankov
Modified: 2008-01-22 10:27 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
Patch for bug 491210 in gtkhtml (1.01 KB, patch)
2008-01-22 04:40 UTC, Bharath Acharya
committed Details | Review

Description Yuri Pankov 2007-10-28 22:27:51 UTC
Please describe the problem:
Pressing 'Close' button in find dialog while composing reply not only closes that dialog, but quits Evolution.

Steps to reproduce:
1. Press Ctrl+L (reply)
2. Press Ctrl+F (find)
3. Press 'Close' button in find dialog


Actual results:
Evolution quits

Expected results:
just close find dialog

Does this happen every time?
yes

Other information:
Comment 1 André Klapper 2007-10-28 22:48:20 UTC
Thanks for taking the time to report this bug.
Without a stack trace from the crash it's very hard to determine what caused it.
Can you get us a stack trace? Please see http://live.gnome.org/GettingTraces for more information on how to do so. Thanks in advance!
Comment 2 Yuri Pankov 2007-10-29 00:20:29 UTC
Sorry, I didn't thought that it's actually a crash until I've run Evolution in gdb. Here's backtrace:

  • #0 search_dialog_response
    at search.c line 98
  • #1 IA__g_cclosure_marshal_VOID__INT
    at gmarshal.c line 216
  • #2 IA__g_closure_invoke
    at gclosure.c line 490
  • #3 signal_emit_unlocked_R
    at gsignal.c line 2440
  • #4 IA__g_signal_emit_valist
  • #5 IA__g_signal_emit
    at gsignal.c line 2243
  • #6 IA__gtk_dialog_response
    at gtkdialog.c line 876
  • #7 action_widget_activated
    at gtkdialog.c line 575
  • #8 IA__g_cclosure_marshal_VOID__VOID
    at gmarshal.c line 77
  • #9 IA__g_closure_invoke
    at gclosure.c line 490
  • #10 signal_emit_unlocked_R
    at gsignal.c line 2440
  • #11 IA__g_signal_emit_valist
  • #12 IA__g_signal_emit
    at gsignal.c line 2243
  • #13 IA__gtk_button_clicked
    at gtkbutton.c line 889
  • #14 gtk_real_button_released
    at gtkbutton.c line 1484
  • #15 IA__g_cclosure_marshal_VOID__VOID
    at gmarshal.c line 77
  • #16 g_type_class_meta_marshal
    at gclosure.c line 567
  • #17 IA__g_closure_invoke
    at gclosure.c line 490
  • #18 signal_emit_unlocked_R
    at gsignal.c line 2370
  • #19 IA__g_signal_emit_valist
  • #20 IA__g_signal_emit
    at gsignal.c line 2243
  • #21 IA__gtk_button_released
    at gtkbutton.c line 881
  • #22 gtk_button_button_release
    at gtkbutton.c line 1377
  • #23 _gtk_marshal_BOOLEAN__BOXED
    at gtkmarshalers.c line 84
  • #24 g_type_class_meta_marshal
    at gclosure.c line 567
  • #25 IA__g_closure_invoke
    at gclosure.c line 490
  • #26 signal_emit_unlocked_R
    at gsignal.c line 2478
  • #27 IA__g_signal_emit_valist
    at gsignal.c line 2209
  • #28 IA__g_signal_emit
    at gsignal.c line 2243
  • #29 gtk_widget_event_internal
    at gtkwidget.c line 4675
  • #30 IA__gtk_widget_event
    at gtkwidget.c line 4475
  • #31 IA__gtk_propagate_event
    at gtkmain.c line 2319
  • #32 IA__gtk_main_do_event
    at gtkmain.c line 1539
  • #33 gdk_event_dispatch
    at gdkevents-x11.c line 2351
  • #34 g_main_dispatch
    at gmain.c line 2061
  • #35 IA__g_main_context_dispatch
    at gmain.c line 2613
  • #36 g_main_context_iterate
    at gmain.c line 2694
  • #37 IA__g_main_loop_run
    at gmain.c line 2898
  • #38 bonobo_main
    from /usr/local/lib/libbonobo-2.so.0
  • #39 main
    at main.c line 602

Comment 3 Srinivasa Ragavan 2008-01-12 12:58:24 UTC
(evolution:31802): GLib-GObject-WARNING **: g_object_weak_unref: couldn't find weak ref 0x5bcf270(0x7a9ea50)
==31802== 
==31802== Invalid read of size 4
==31802==    at 0xA8D9E93: search_dialog_response (search.c:98)
==31802==    by 0x5653EA4: g_cclosure_marshal_VOID(i_xx_t) (gmarshal.c:216)
==31802==    by 0x56459A1: g_closure_invoke (gclosure.c:490)
==31802==    by 0x565AB6C: signal_emit_unlocked_R (gsignal.c:2440)
==31802==    by 0x565C707: g_signal_emit_valist (gsignal.c:2199)
==31802==    by 0x565CB54: g_signal_emit (gsignal.c:2243)
==31802==    by 0x41B8A20: gtk_dialog_response (gtkdialog.c:903)
==31802==    by 0x41B8A64: action_widget_activated (gtkdialog.c:602)
==31802==    by 0x565323E: g_cclosure_marshal_VOID__VOID (gmarshal.c:77)
==31802==    by 0x56459A1: g_closure_invoke (gclosure.c:490)
==31802==    by 0x565AB6C: signal_emit_unlocked_R (gsignal.c:2440)
==31802==    by 0x565C707: g_signal_emit_valist (gsignal.c:2199)
==31802==  Address 0xB4FE370 is 24 bytes inside a block of size 28 free'd
==31802==    at 0x402243F: free (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so)
==31802==    by 0x56BA6C0: g_free (gmem.c:187)
==31802==    by 0xA8D9677: gtk_html_search_dialog_destroy (search.c:164)
==31802==    by 0xA8D9E89: search_dialog_response (search.c:97)
==31802==    by 0x5653EA4: g_cclosure_marshal_VOID(i_xx_t) (gmarshal.c:216)
==31802==    by 0x56459A1: g_closure_invoke (gclosure.c:490)
==31802==    by 0x565AB6C: signal_emit_unlocked_R (gsignal.c:2440)
==31802==    by 0x565C707: g_signal_emit_valist (gsignal.c:2199)
==31802==    by 0x565CB54: g_signal_emit (gsignal.c:2243)
==31802==    by 0x41B8A20: gtk_dialog_response (gtkdialog.c:903)
==31802==    by 0x41B8A64: action_widget_activated (gtkdialog.c:602)
==31802==    by 0x565323E: g_cclosure_marshal_VOID__VOID (gmarshal.c:77)



The above log may give a better reason for this.
Comment 4 Bharath Acharya 2008-01-22 04:40:36 UTC
Created attachment 103396 [details] [review]
Patch for bug 491210 in gtkhtml
Comment 5 Srinivasa Ragavan 2008-01-22 05:30:19 UTC
Looks fine to commit. I tested with valgrind also, it works great. Thanks Bharath.
Comment 6 Suman Manjunath 2008-01-22 10:27:53 UTC
Committed to SVN trunk as r8694
(http://svn.gnome.org/viewvc/gtkhtml?view=revision&revision=8694)