[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    [Bug 113329] KMail crash on startup
From:       Will Stephenson <wstephenson () kde ! org>
Date:       2007-03-20 18:47:10
Message-ID: 20070320184710.16786.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=113329         




------- Additional Comments From wstephenson kde org  2007-03-20 19:47 -------
SVN commit 644705 by wstephens:

Don't continue checking mail in the other folders when a local folder was removed, it \
causes a crash on startup.

There is still a bug that local folders are removed erroneously because their \
ImapPath somehow become empty, but at least they can be recovered with Refresh Folder \
List now. CCBUG:113329


 M  +3 -1      kmacctimap.cpp  


--- branches/work/kmail_debug_imap/kmacctimap.cpp #644704:644705
 @ -342,7 +342,7  @
         else {
           connect(imapFolder, SIGNAL(numUnreadMsgsChanged(KMFolder*)),
               this, SLOT(postProcessNewMail(KMFolder*)));
-          bool ok = imapFolder->processNewMail(interactive);
+          bool ok = imapFolder->processNewMail(interactive); // this removes the \
local kmfolderimap if its imapPath is somehow empty, and removing it calls \
createFolderList, invalidating mMailCheckFolders, and causing a crash  if (!ok)
           {
             // there was an error so cancel
 @ -352,6 +352,8  @
               mMailCheckProgressItem->incCompletedItems();
               mMailCheckProgressItem->updateProgress();
             }
+            // since the list of folders might have been updated at this point, \
mMailCheckFolders may be invalid, so break +            break;
           }
         }
       }
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic