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 349967 - Evolution crashes if gconf default window sizes are corrupted
Evolution crashes if gconf default window sizes are corrupted
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Shell
2.8.x (obsolete)
Other All
: Normal critical
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2006-08-04 17:52 UTC by Daniel Gryniewicz
Modified: 2013-09-13 00:51 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16

Attachments
Proposed fix (1.32 KB, patch)
2006-08-04 17:53 UTC, Daniel Gryniewicz 		committed Details | Review

Description Daniel Gryniewicz 2006-08-04 17:52:27 UTC 
Steps to reproduce:
My default height and width in gconf somehow got set to -bignum, and so evo was
crashing on startup.  The only way to stop this was to edit gconf, and change the settings, which an end user could not do. 

Versions:
evolution 2.7.90
evolution-data-server 1.7.90.1
gtk+ 2.10.1
glib 2.12.1


Stack trace:

+ Trace 69979

  • #0 IA__g_logv
    at g messages.c line 493
  • #2 IA__g_return_if_fail_warning
    at gmessages.c line 532
  • #3 IA__gtk_window_set_default_size
    at gtkwindow.c line 3478
  • #4 e_shell_window_new
    at e-shell-window.c line 1047
  • #5 e_shell_create_window
    at e-shell.c line 933
  • #6 idle_cb
    at main.c line 380
  • #7 g_idle_dispatch
    at gmain.c line 3924
  • #8 g_main_dispatch
    at gmain.c line 2043
  • #9 IA__g_main_context_dispatch
    at gmain.c line 2594
  • #10 g_main_context_iterate
    at gmain.c line 2675
  • #11 IA__g_main_loop_run
    at gmain.c line 2879
  • #12 bonobo_main
    from /usr/lib/libbonobo-2.so.0
  • #13 main
    at main.c line 614 


Other information:
My fix is to sanity check the width and height from gconf before passing them
in.  Patch attached.
Comment 1 Daniel Gryniewicz 2006-08-04 17:53:27 UTC 
Created attachment 70214 [details] [review]
Proposed fix
Comment 2 Daniel Gryniewicz 2006-08-10 01:40:38 UTC 
bug #350678 is apparently the cause of the sizes being messed up.
Comment 3 Srinivasa Ragavan 2006-08-22 10:28:36 UTC 
looks fine to commit.