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 359236 - Evolution 2.8.1 crashes on startup
Evolution 2.8.1 crashes on startup
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.8.x (obsolete)
Other Linux
: Normal critical
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 359520 361484 361511 361521 362997 367176 401842 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-10-03 10:39 UTC by Matthew Barnes
Modified: 2007-05-02 09:07 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Backtrace (20.09 KB, text/plain)
2006-10-03 10:47 UTC, Matthew Barnes 		  Details
Proposed patch (675 bytes, patch)
2006-10-04 06:38 UTC, Srinivasa Ragavan 		committed Details | Review

Description Matthew Barnes 2006-10-03 10:39:32 UTC 
Evolution 2.8.1 segfaulted on me while switching to an IMAP folder.  Now I'm getting the same crash everytime I try to start Evolution.  See the backtrace below.

A quick GDB session showed suspicious behavior in e-filter-bar.c:set_property(), when setting PROP_STATE:

    view_id = xml_get_prop_int (node, "view_id");
    scope = xml_get_prop_int (node, "searchscope");
    item_id = xml_get_prop_int (node, "item_id");

I'm getting -1 for all three of these variables, which suggests that the XML properties are not present in the search state string.  But the logic assumes they are and carries on to use item_id (-1) as an array index, which triggers the segmentation fault.

I'm not sure if this is the root problem, though.
Comment 1 Matthew Barnes 2006-10-03 10:47:13 UTC 
Created attachment 73932 [details]
Backtrace
Comment 2 Matthew Barnes 2006-10-03 20:02:32 UTC 
The offending code was added just recently:
http://cvs.gnome.org/viewcvs/evolution/widgets/misc/e-filter-bar.c?r1=1.57&r2=1.58

CC'ing Srini
Comment 3 Srinivasa Ragavan 2006-10-04 06:15:19 UTC 
Now evolution restores the old save search on per folder basis. Im not able to reproduce it. Could be possible that your cmeta file has some cfg, causing the crash. Could be great if you can attach that particular cmeta file. or atleast the search state in folder_name.cmeta.
Comment 4 Srinivasa Ragavan 2006-10-04 06:35:23 UTC 
Got it. Last time the search was saved, it didnt have these three. (2 are new) The code expected them to be there. *sigh. Can you try this patch Mathew? Thanks for the bug.
Comment 5 Srinivasa Ragavan 2006-10-04 06:38:03 UTC 
Created attachment 73991 [details] [review]
Proposed patch
Comment 6 Matthew Barnes 2006-10-04 09:28:23 UTC 
Srini, the patch seems to fix it.  Thanks for the quick turnaround.
Comment 7 Srinivasa Ragavan 2006-10-04 12:36:02 UTC 
Will commit it tomorrow.
Comment 8 Srinivasa Ragavan 2006-10-11 08:27:51 UTC 
Fixed to 2.8
Comment 9 Harish Krishnaswamy 2006-10-11 11:16:36 UTC 
*** Bug 359520 has been marked as a duplicate of this bug. ***
Comment 10 Karsten Bräckelmann 2006-10-11 22:32:06 UTC 
*** Bug 361484 has been marked as a duplicate of this bug. ***
Comment 11 Karsten Bräckelmann 2006-10-12 23:03:23 UTC 
*** Bug 361521 has been marked as a duplicate of this bug. ***
Comment 12 Karsten Bräckelmann 2006-10-12 23:10:46 UTC 
*** Bug 361511 has been marked as a duplicate of this bug. ***
Comment 13 Karsten Bräckelmann 2006-10-19 08:52:04 UTC 
*** Bug 362997 has been marked as a duplicate of this bug. ***
Comment 14 André Klapper 2006-11-08 17:35:21 UTC 
*** Bug 367176 has been marked as a duplicate of this bug. ***
Comment 15 Akhil Laddha 2007-05-02 09:07:53 UTC 
*** Bug 401842 has been marked as a duplicate of this bug. ***