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 360237 - Trash does not show deleted messages, results in broken inbox, crashes application.
Trash does not show deleted messages, results in broken inbox, crashes applic...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.8.x (obsolete)
Other Linux
: High critical
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 359678 359774 359785 359830 359931 359957 360097 360122 360238 360284 360308 360501 361060 361112 361160 361185 361302 361307 361344 361414 361474 361782 361831 362716 367560 369496 377761 384865 384866 388049 471855 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-10-06 18:25 UTC by Adam McMaster
Modified: 2013-09-10 14:04 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Stack trace 1 (103.56 KB, text/plain)
2006-10-06 18:26 UTC, Adam McMaster
  Details
Stack trace 2 (9.39 KB, text/plain)
2006-10-06 18:28 UTC, Adam McMaster
  Details
Stack trace 3 (26.87 KB, text/plain)
2006-10-06 18:30 UTC, Adam McMaster
  Details
Stack trace 4 (11.89 KB, text/plain)
2006-10-06 18:33 UTC, Adam McMaster
  Details
Revert part of latest commit in mail/em-folder-brower.c (857 bytes, patch)
2006-10-06 22:08 UTC, Jan de Groot
none Details | Review
Tryout patch (1.59 KB, patch)
2006-10-07 18:43 UTC, Srinivasa Ragavan
committed Details | Review

Description Adam McMaster 2006-10-06 18:25:20 UTC
Clicking the Trash mailbox does not display any messages, despite the window title reporting "Trash ([x] deleted)" where [x] is the correct number of messages.  Instead the message list is completely empty.

After viewing the trash and returning to the inbox, an error message is displayed (see below).  After closing this error message, the inbox also now displays an empty message list.  This continues to happen for the inbox for that particular account until evolution is closed.

The error message:



Error while Generating message list.

Error executing search expression: Unknown type in parse tree: 140585792:
(and (match-all (and (not (system-flag "deleted")) (not (system-flag "junk"))))
  )



End error message.

Evolution will then crash if "Hide deleted messages" is enabled or disabled (it does not matter which).  It also seems to crash if this option is enabled or disabled before triggering the above bug (i.e. the user has not closed Evolution since changing that option).

I have a total of four stack traces for this bug, each generated from Bug Buddy after triggering the crash in slightly different ways.  I will attach them all in a moment.
Comment 1 Adam McMaster 2006-10-06 18:26:24 UTC
Created attachment 74169 [details]
Stack trace 1

This stack trace was generated by viewing the trash, then viewing the inbox, closing the error message window and then disabling "Hide deleted messages" from the View menu.
Comment 2 Adam McMaster 2006-10-06 18:28:00 UTC
Created attachment 74170 [details]
Stack trace 2

This stack trace was generated when viewing the Trash while a new message was being filtered.  As soon as the message was done being filtered evo crashed.  I had not yet viewed the inbox (or any other folders).
Comment 3 Adam McMaster 2006-10-06 18:30:09 UTC
Created attachment 74171 [details]
Stack trace 3

This was the same as trace 1, but enabling "Hide deleted messages" instead of disabling it.
Comment 4 Adam McMaster 2006-10-06 18:33:45 UTC
Created attachment 74173 [details]
Stack trace 4

This was generated by first enabling "Hide deleted messages", then viewing the trash, then viewing the inbox.  If I remember correctly, evo crashed as soon as the error window was closed.
Comment 5 André Klapper 2006-10-06 22:06:17 UTC
i have a strange feeling that we might have many, many duplicates of this one here. srini: poke?
Comment 6 Jan de Groot 2006-10-06 22:08:11 UTC
Created attachment 74186 [details] [review]
Revert part of latest commit in mail/em-folder-brower.c

This patch reverts the problematic part of the latest commit to em-folder-browser.c. I have no idea what this does or what it fixes, but it makes the trashfolder problem go away. This patch is not intended as a fix, more as a pointer to the problem.
Comment 7 Srinivasa Ragavan 2006-10-07 17:43:12 UTC
Seems like a search bug in my last commit. Ill look into it.
Comment 8 Srinivasa Ragavan 2006-10-07 17:48:14 UTC
Hmm Got it. It tries to save the state for Trash which is a vfolder and doesnt seem to have settings for it. Ill put up a patch for it soon.
Comment 9 Srinivasa Ragavan 2006-10-07 18:19:37 UTC
Just did a quick look into it. Seems to have the same issue for junk folder too. The save/retrieve search for both seem to have some issue
Comment 10 Srinivasa Ragavan 2006-10-07 18:43:10 UTC
Created attachment 74229 [details] [review]
Tryout patch
Comment 11 Srinivasa Ragavan 2006-10-07 18:44:41 UTC
Can you try this? This should be the main cause. Just the last hunk of this patch should fix and the other two are for some other issues. Im yet to verify the fix on other complicated scenarios.
Comment 12 Jan de Groot 2006-10-07 19:05:15 UTC
I can confirm this patch works. The error message on viewing inbox after trash about the bad search filter is gone, the crash after entering a searchfilter in the trashfolder is also gone and the contents of the trashfolder are shown correctly now.
Comment 13 Srinivasa Ragavan 2006-10-07 19:59:25 UTC
Thx Jan. Ill close on this. soon. The final patch might look little different. Ill fix and upload sometime this week.
Comment 14 Karsten Bräckelmann 2006-10-10 12:05:42 UTC
Pasting some part of stacktrace 4, omitting some libc foo. Also removing some string noise.

  • #13 pthread_mutex_unlock
    from /lib/tls/i686/cmov/libc.so.6
  • #14 e_thread_put
    at e-msgport.c line 1148
  • #15 ml_regen_timeout
    at message-list.c line 3923
  • #16 message_list_thaw
    at message-list.c line 3305
  • #17 emfb_set_folder
    at em-folder-browser.c line 1899
  • #18 emfv_got_folder
    at em-folder-view.c line 691
  • #19 get_folder_got
    at mail-ops.c line 1230
  • #20 mail_msgport_replied
    at mail-mt.c line 461

Comment 15 Karsten Bräckelmann 2006-10-10 12:10:02 UTC
Srini, I believe bug 359785 and its 6 duplicates to be the very same crash.

Andre, got the same feeling. Most likely a lot of the recent useless stacktraces, closed INCOMPLETE, actually are duplicates too.
Comment 16 Karsten Bräckelmann 2006-10-10 12:10:42 UTC
*** Bug 359785 has been marked as a duplicate of this bug. ***
Comment 17 Karsten Bräckelmann 2006-10-10 12:13:55 UTC
*** Bug 361060 has been marked as a duplicate of this bug. ***
Comment 18 Karsten Bräckelmann 2006-10-10 12:15:34 UTC
*** Bug 361112 has been marked as a duplicate of this bug. ***
Comment 19 Karsten Bräckelmann 2006-10-10 12:17:14 UTC
*** Bug 359931 has been marked as a duplicate of this bug. ***
Comment 20 Karsten Bräckelmann 2006-10-10 12:18:00 UTC
*** Bug 359830 has been marked as a duplicate of this bug. ***
Comment 21 Karsten Bräckelmann 2006-10-10 12:23:29 UTC
*** Bug 359957 has been marked as a duplicate of this bug. ***
Comment 22 Stephane Loeuillet 2006-10-10 12:30:46 UTC
if bug #359957 is really a dupe of this one, this has nothing to do with IMAP as I use POP3 exclusively and never used IMAP4.

So the bug title should be changed (-"[IMAP] ")
Comment 23 Srinivasa Ragavan 2006-10-10 12:59:03 UTC
Guenther, The attached patch should fix the problem as said in comment #12 . Im looking into few more scenarios. Ill get this in asap.
Comment 24 André Klapper 2006-10-10 13:29:02 UTC
i can reproduce this with local folders, right.
Comment 25 Germán Poo-Caamaño 2006-10-10 15:03:06 UTC
*** Bug 361160 has been marked as a duplicate of this bug. ***
Comment 26 Germán Poo-Caamaño 2006-10-10 15:05:33 UTC
I marked 361160 as duplicated of this one, but I'm able to reproduce with the Junk folder instead of Trash.  An stack trace is attached in that bug, if it helps.
Comment 27 Germán Poo-Caamaño 2006-10-10 15:08:00 UTC
*** Bug 359774 has been marked as a duplicate of this bug. ***
Comment 28 Karsten Bräckelmann 2006-10-10 17:38:11 UTC
*** Bug 361185 has been marked as a duplicate of this bug. ***
Comment 29 Karsten Bräckelmann 2006-10-10 17:43:44 UTC
(In reply to comment #23)
> Guenther, The attached patch should fix the problem as said in comment #12 . Im
> looking into few more scenarios. Ill get this in asap.

Srini, I never doubted that. :)  I merely poked you to have a close look at the duplicates and the really different stacktraces attached to this bug.

Anyway, comment 22 and comment 24 kind of already took care about that. Thanks Stephane for caring and pointing this out.
Comment 30 Srinivasa Ragavan 2006-10-10 17:55:53 UTC
ok :) I got it. This isnt specific to any provider. It can happen on any folder to be frank (junk, trash, etc)
Comment 31 Karsten Bräckelmann 2006-10-10 19:45:16 UTC
*** Bug 360284 has been marked as a duplicate of this bug. ***
Comment 32 Karsten Bräckelmann 2006-10-10 19:47:08 UTC
Although slightly different (notably the emfb_search_search_activated() call), I believe bug 360284 to be yet another duplicate of this very crash...
Comment 33 Karsten Bräckelmann 2006-10-11 00:29:24 UTC
*** Bug 361302 has been marked as a duplicate of this bug. ***
Comment 34 Srinivasa Ragavan 2006-10-11 08:27:30 UTC
Fixed to 2.8
Comment 35 André Klapper 2006-10-11 11:55:30 UTC
*** Bug 360501 has been marked as a duplicate of this bug. ***
Comment 36 André Klapper 2006-10-11 11:56:04 UTC
*** Bug 360308 has been marked as a duplicate of this bug. ***
Comment 37 André Klapper 2006-10-11 11:56:14 UTC
*** Bug 359678 has been marked as a duplicate of this bug. ***
Comment 38 Karsten Bräckelmann 2006-10-11 16:03:08 UTC
*** Bug 361344 has been marked as a duplicate of this bug. ***
Comment 39 Karsten Bräckelmann 2006-10-13 16:03:29 UTC
*** Bug 360238 has been marked as a duplicate of this bug. ***
Comment 40 Karsten Bräckelmann 2006-10-13 16:05:46 UTC
*** Bug 361831 has been marked as a duplicate of this bug. ***
Comment 41 Karsten Bräckelmann 2006-10-13 16:12:47 UTC
GAR.

Srini, would you please have a second look at this patch and the one committed to fix bug 359236? This one got both...

Although I am pleased to see these nasty and often reported crashes being fixed, I wonder, how you could commit both. Also, you just messed with my clean track of duplicates. ;)


Seriously, thanks for fixing these little buggers, Sini. :)
This will cut down on the flood of useless stacktraces.
Comment 42 Srinivasa Ragavan 2006-10-13 16:24:10 UTC
Guenther, I didnt get you. Im sure these patches fixes the bug. 
Comment 43 Christian Hamar 2006-10-13 16:58:28 UTC
Uhm. I noticed one more problem. Bug 361831 <- this duplication bug. I repoted that then i noticed that in this bug there is a patch for it.

Nah, i applied this patch to my 2.8.1 evolution and segfaults and dialog errors disappeared. But seems that Trash does not operates full correctly.

I mean a simple example:

I got 10 mail. I delete 3 mail. Those are in Trash now. I can see them.

I quit evolution and start it again. I got to Trash, and i see only 1 message. After clicking to inbox and other dirs and click back to Trash i see ALL 3 mail again. :S This is intresting and dunno what cause this.

Any idea about ?

Regards

Christian Hamar alias krix
Hungary
Comment 44 Srinivasa Ragavan 2006-10-13 17:13:10 UTC
I dont think this has anything to do with Search fixes or implementation. Any similiar observations? Im puzzled. I doubt this may be something to do with the folder yet to be opened. Mail hackers may have an answer for it.
Comment 45 André Klapper 2006-10-17 10:18:01 UTC
*** Bug 362716 has been marked as a duplicate of this bug. ***
Comment 46 Karsten Bräckelmann 2006-10-18 16:39:16 UTC
*** Bug 361414 has been marked as a duplicate of this bug. ***
Comment 47 Karsten Bräckelmann 2006-10-18 16:41:21 UTC
*** Bug 361474 has been marked as a duplicate of this bug. ***
Comment 48 Karsten Bräckelmann 2006-10-19 11:23:15 UTC
*** Bug 361782 has been marked as a duplicate of this bug. ***
Comment 49 Karsten Bräckelmann 2006-10-23 22:32:36 UTC
*** Bug 361307 has been marked as a duplicate of this bug. ***
Comment 50 André Klapper 2006-10-27 23:42:24 UTC
*** Bug 360097 has been marked as a duplicate of this bug. ***
Comment 51 André Klapper 2006-10-27 23:43:23 UTC
*** Bug 360122 has been marked as a duplicate of this bug. ***
Comment 52 André Klapper 2006-11-08 17:41:33 UTC
*** Bug 367560 has been marked as a duplicate of this bug. ***
Comment 53 André Klapper 2006-11-21 14:33:44 UTC
*** Bug 377761 has been marked as a duplicate of this bug. ***
Comment 54 palfrey 2007-04-11 22:28:58 UTC
*** Bug 384866 has been marked as a duplicate of this bug. ***
Comment 55 palfrey 2007-04-11 22:29:04 UTC
*** Bug 369496 has been marked as a duplicate of this bug. ***
Comment 56 palfrey 2007-04-11 22:29:42 UTC
*** Bug 388049 has been marked as a duplicate of this bug. ***
Comment 57 palfrey 2007-04-11 22:30:42 UTC
*** Bug 384865 has been marked as a duplicate of this bug. ***
Comment 58 palfrey 2007-08-30 16:50:45 UTC
*** Bug 471855 has been marked as a duplicate of this bug. ***