See top of individual patches for detailed descriptions. Reproducible: Always Steps to Reproduce:
Created attachment 113648 [details, diff] kmail-3.5.6-crash-unfixed-filter-mails.patch http://bugs.debian.org/404032 -> http://bugs.kde.org/139886 -> http://bugs.kde.org/128593 -> http://bugs.kde.org/108971 Status: Still not merged upstream Bug#404032: kmail: crash when filtering mails into current folder Michael Eyrich Tue, 06 Feb 2007 08:26:00 -0800 Package: kmail Version: 4:3.5.5.dfsg.1-6 Followup-For: Bug #404032 It looks like kmail does not honour multiple opens of the same mailbox thus closing the current mailbox unmindful and crashes. the attached (dirty) patch avoids closing a folder if already open.
Created attachment 113652 [details, diff] kmail-3.5.6-debian-18_optimize_splitAddressInternal.diff http://bugs.kde.org/137033 : we have been shipping this patch in debian for a couple of months. We have gotten one report of some drawbacks with it, but I still consider them less imp$ A user writes: <quote> Since version 4:3.5.5.dfsg.1-2 kmail stopped treating correctly my name in the header. The last à in my last name is lost (strangely enough if I write it not in the last position of my name in the profile it works without problem). Prior to that version it worked without problem. I suspect the culprit is 18_optimize_splitAddressInternal.diff </quote> Original report is http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=405867 /Sune
Created attachment 113654 [details, diff] kmail-3.5.6-imap-crash-svn-631934.patch http://websvn.kde.org/?view=rev&revision=631934 SVN commit 631934 by dfaure: Found the reason why KMAcctCachedImap::processNewMail's assert below was hit assert( mFolder ); // George says "better to crash then lose mail" The configuration dialog creates temporary copies of the account when modifying one; but it doesn't delete that copy when closing with Cancel. What's more, the copied account started its own timer for interval-mail-checking, and when that timer fired, KMAcctCachedImap would crash due to mFolder=0. So: 1) delete things in the dtor to avoid the account memleak 2) don't start the mailcheck timer when simply setting the checkinterval of an account, only when the account is added to the AccountManager for real. BUG: 117935 http://bugs.kde.org/117935
Created attachment 113657 [details, diff] kmail-3.5.6-imap-crash-svn-643329.patch http://websvn.kde.org/?view=rev&revision=643329 Revision 643329 - (view) (download) (as text) (annotate) - [select for diffs] Modified Fri Mar 16 21:54:53 2007 UTC (36 hours, 51 minutes ago) by coolo File length: 77033 byte(s) Diff to previous 643194 fix the recovery path patch - meanwhile we think we know Revision 643194 - (view) (download) (as text) (annotate) - [select for diffs] Modified Fri Mar 16 14:40:35 2007 UTC (44 hours, 6 minutes ago) by coolo File length: 77029 byte(s) Diff to previous 571100 Till says my patch looks correct for the recovery path CCBUG: 126715 http://bugs.kde.org/126715
Created attachment 113658 [details, diff] kmail-3.5.6-crash-svn-643369.patch http://websvn.kde.org/?view=rev&revision=643369 Revision 643369 - (view) (download) (as text) (annotate) - [select for diffs] Modified Sat Mar 17 02:16:46 2007 UTC (32 hours, 41 minutes ago) by winterz File length: 11631 byte(s) Diff to previous 614050 backport SVN commit 643168 by coolo: this looks crashy (and I guess this is the same with 3.5) (CID 3205)
Created attachment 113659 [details] kmail-3.5.6-r2.ebuild Ebuild for the patches
Created attachment 113660 [details] kmail.tar.bz2 Tar of my kmail directory to be untarred under kde-base in case I forgot anything - I will be working 24/7 the next week or so.
(In reply to comment #6) > Created an attachment (id=113659) [edit] > kmail-3.5.6-r2.ebuild > > Ebuild for the patches > Please attach patches to ebuilds, not whole ebuilds. Also please create the appropriate patch to kdepim ebuild.
(In reply to comment #8) > (In reply to comment #6) > > Created an attachment (id=113659) [edit] > > kmail-3.5.6-r2.ebuild > > > > Ebuild for the patches > > > > Please attach patches to ebuilds, not whole ebuilds. Gotcha. Hmm... That actually makes sense, now I think about it. > Also please create the appropriate patch to kdepim ebuild. Done. I can't build-test it for obvious reasons.
Created attachment 113703 [details, diff] kmail-3.5.6-r1.patch Patch against kmail-3.5.6-r1.ebuild.
Created attachment 113704 [details, diff] kdepim-3.5.6-r1.patch Patch against kdepim-3.5.6-r1.ebuild.
Created attachment 115351 [details, diff] kmail-3.5.6-fixes.diff I'm not fond to include stuff that isn't in the 3.5 branch, especially if it attempts to solve a minor problem, as the second patch. Would you test the attached patch, please?! It includes basically all branch bug fixes, but not the new features. Simply replacing the PATCHES=... line with the patch should suffice. I don't use imap accounts, so this patch might even eat your kitty - hope not, though. ;)
Created attachment 115548 [details, diff] kmail-3.5.6-fixes.diff minor changes to the above patch
Please give kmail-3.5.6-r2 a try. Considering this bug fixed.
