GNOME Bugzilla – Bug 87222
segfault starting up
Last modified: 2004-12-22 21:47:04 UTC
Package: GnomeICU Severity: normal Version: 1.89.106 Synopsis: segfault starting up Bugzilla-Product: GnomeICU Bugzilla-Component: Main Window BugBuddy-GnomeVersion: 2.0 (2.0.1) Description: Description of Problem: When I try to start gnomeicu I get a segfault. Steps to reproduce the problem: 1. 2. 3. Actual Results: Expected Results: How often does this happen? Additional Information: Debugging Information: Backtrace was generated from '/usr/local/bin/gnomeicu' [New Thread 1024 (LWP 10171)] [New Thread 2049 (LWP 10172)] 0x40a92019 in wait4 () from /lib/libc.so.6
+ Trace 24635
------- Bug moved to this database by unknown@bugzilla.gnome.org 2002-07-03 05:00 ------- Unknown version unspecified in product GnomeICU. Setting version to "0.94.1". Reassigning to the default owner of the component, jwise@springthrough.com.
Found this bug. In get_foreground_for_status() there is a test for various statuses to get a color for them. It was missing a case for STATUS_NA_99A. This patch fixes it although I wonder how many other places need a case for STATUS_NA_99A and/or statuses not being tested for. Index: src/gtkfunc.c =================================================================== RCS file: /cvs/gnome/gnomeicu/src/gtkfunc.c,v retrieving revision 1.314 diff -u -r1.314 gtkfunc.c --- src/gtkfunc.c 1 Jul 2002 00:22:12 -0000 1.314 +++ src/gtkfunc.c 3 Jul 2002 18:10:48 -0000 @@ -1680,6 +1680,7 @@ color = &color_away; break; case STATUS_NA: + case STATUS_NA_99A: color = &color_na; break; case STATUS_OCCUPIED: A default case or a test that color actually got assigned something is probably in order to trap for this kind of error in the future.
I think this has been fixed
Well, a default case has been added to the switch() in the particular function mentioned above, but there was no case for STATUS_NA_99A added. It seems that there is at least one client (gaim I think) that is reporting this as it's NA status. Should gnomeicu not handle STATUS_NA_99A in all places where status is tested for various reasons?
*** Bug 88428 has been marked as a duplicate of this bug. ***
Better stack trace in bug 88428