GNOME Bugzilla – Bug 332807
Copy contact to address book causes Evolution to crash.
Last modified: 2006-03-01 13:03:22 UTC
Steps to reproduce: 1. Launch Evolution on Solaris -> Goto "Contact" 2. Right click -> Choose "New Contact List.." 3. Create a contact: -> Click "Add" -> OK. e.g: Name: abc Email: abc@abc.COM 4. Goto new created contact -> Right click on it. 5. Choose "Copy to Address Book..." Stack trace: 2821: /usr/lib/evolution-2.6 ----------------- lwp# 1 / thread# 1 -------------------- d1ba93d5 waitid (0, b07, 8045db0, 3) d1b9ccab waitpid (b07, 8045ec4, 0) + 70 d25713da libgnomeui_segv_handle (5, 0, 8045f70) + aa d1ba7a8f __sighndlr (5, 0, 8045f70, d2571330) + f d1b9d814 call_user_handler (5, 0, 8045f70) + 247 d1b9d99c sigacthandler (5, 0, 8045f70) + bc --- called from signal handler with signal 5 (SIGTRAP) --- d1ba8fa5 _lwp_kill (1, 5) + 15 d1b6dd14 raise (5) + 22 d1c311b5 g_logv (0, 8, d1aedbe8, 8046658) + 299 d1c3127d g_log (0, 8, d1aedbe8, d1aedbbc, 6d, d1aedbdc) + 29 d1ae4781 html_utils_get_accessible (0, 8449e28) + 155 d1ae0e7f gtk_html_a11y_new (8336a30) + f3 d1adf163 gtk_html_a11y_factory_create_accessible (8336a30) + 3f d171af6b atk_object_factory_create_accessible (849a230, 8336a30) + 4b d214ae62 gtk_widget_real_get_accessible (8336a30) + 46 d214adb5 gtk_widget_get_accessible (8336a30) + 39 d24e883c glade_xml_add_accessibility_info (844f6c0, 8336a30, 8483a18) + 20 d24ea9ae glade_xml_set_common_params (844f6c0, 8336a30, 8483a18) + 1b6 d24ea3d5 glade_xml_build_widget (844f6c0, 8483a18) + 289 d24ea06d glade_standard_build_children (844f6c0, 84869a0, 845f9b0) + 85 d24eaba0 glade_xml_set_common_params (844f6c0, 84869a0, 845f9b0) + 3a8 d24ea3d5 glade_xml_build_widget (844f6c0, 845f9b0) + 289 d24ea06d glade_standard_build_children (844f6c0, 8486a00, 84912b0) + 85 d24eaba0 glade_xml_set_common_params (844f6c0, 8486a00, 84912b0) + 3a8 d24ea3d5 glade_xml_build_widget (844f6c0, 84912b0) + 289 d24ea06d glade_standard_build_children (844f6c0, 83f9668, 8492580) + 85 d24eaba0 glade_xml_set_common_params (844f6c0, 83f9668, 8492580) + 3a8 d24ea3d5 glade_xml_build_widget (844f6c0, 8492580) + 289 d24ea06d glade_standard_build_children (844f6c0, 8216b38, 8482af0) + 85 d24eaba0 glade_xml_set_common_params (844f6c0, 8216b38, 8482af0) + 3a8 d24ea694 glade_xml_handle_internal_child (844f6c0, 8336980, 8491b40) + 294 d24ea05b glade_standard_build_children (844f6c0, 8336980, 845ac28) + 73 d24ef88a gtk_dialog_build_children (844f6c0, 8336980, 845ac28) + 3a d24eaba0 glade_xml_set_common_params (844f6c0, 8336980, 845ac28) + 3a8 d24ea3d5 glade_xml_build_widget (844f6c0, 845ac28) + 289 d24e905e glade_xml_build_interface (844f6c0, 847d668, 0) + c6 d24e7c8a glade_xml_construct (844f6c0, 845fab8, 0, 0) + 6e d24e7bf7 glade_xml_new (845fab8, 0, 0) + 37 ce7feb5f match_query_callback (844f968, 83ba3b0, 4, 83fffa0) + 9b ce7fe30b query_cb (849a210, 0, 8493a90, 833b870) + 153 d037538b emit_async_get_contacts_response (8458d58) + 33 d1c2c6ab g_idle_dispatch (844e8c0, d0375358, 8458d58) + 1f d1c29664 g_main_dispatch (809cfa8) + 1c8 d1c2a74d g_main_context_dispatch (809cfa8) + 85 d1c2ab6d g_main_context_iterate (809cfa8, 1, 1, 80824f8) + 3d1 d1c2b172 g_main_loop_run (818f8c0) + 1ba d236f40e bonobo_main (8047310, 80471f8, d27fb840, 8068474, 63, 1) + 5e 08063d19 main (1, 804723c, 8047244) + 6ed 08058fca _start (1, 8047378, 0, 804738f, 80473c9, 804738f) + 7a *** (#1 of 2): 2006-02-24 16:54:43 CST simon.zheng@sun.com *** Last Edit: 2006-02-24 20:26:31 CST simon.zheng@sun.com Other information:
Created attachment 60281 [details] [review] patch
Crash is caused by logging a critical message. When engine is just initilized, clue is possibly NULL. To avoid logging a critical message, check validity before invoking html_utils_get_accessible(). In addition, initilizing a11y object only when accessibility is enabled.
*** Bug 332789 has been marked as a duplicate of this bug. ***
Confirming as per comment 3.
By a quick first test this patch seem to fix bug 332789 for me. Note though, that I needed to rebuild GtkHTML fresh. Adding Srini to the Cc list.
Looks fine simonz.
Thanks, srag, Committed the patch to head.
Please don't forget to set the patch status accordingly.