Hello community, here is the log from the commit of package kdepim3 checked in at Thu Mar 22 20:47:35 CET 2007. -------- --- KDE/kdepim3/kdepim3.changes 2007-03-21 15:27:45.000000000 +0100 +++ /mounts/work_src_done/STABLE/kdepim3/kdepim3.changes 2007-03-22 14:12:28.000000000 +0100 @@ -1,0 +2,5 @@ +Thu Mar 22 14:12:00 CET 2007 - wstephenson@suse.de + +- Fix #152199 - crash when deleting searched messages + +------------------------------------------------------------------- New: ---- 3_5_BRANCH-r645363.diff ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ kdepim3.spec ++++++ --- /var/tmp/diff_new_pack.n10697/_old 2007-03-22 20:47:20.000000000 +0100 +++ /var/tmp/diff_new_pack.n10697/_new 2007-03-22 20:47:20.000000000 +0100 @@ -29,7 +29,7 @@ Group: System/GUI/KDE Summary: Personal Information Manager Software for KDE Version: 3.5.6 -Release: 13 +Release: 14 URL: http://www.kde.org/ Requires: kdelibs3 >= %( echo `rpm -q --queryformat '%{VERSION}' kdelibs3` ) Requires: kdebase3 >= %( echo `rpm -q --queryformat '%{VERSION}' kdelibs3` ) @@ -56,6 +56,7 @@ Patch27: disable-kitchensync.diff Patch28: 3_5_BRANCH-r635257.diff Patch29: kmail-stability-fixes-r644741:644903.diff +Patch30: 3_5_BRANCH-r645363.diff Provides: cryptplug Obsoletes: cryptplug %if %suse_version > 1000 @@ -213,6 +214,7 @@ %patch27 %patch28 %patch29 +%patch30 . /etc/opt/kde3/common_options update_admin @@ -687,6 +689,8 @@ /opt/kde3/%_lib/kde3/plugins/designer/kpartsdesignerplugin.* %changelog +* Thu Mar 22 2007 - wstephenson@suse.de +- Fix #152199 - crash when deleting searched messages * Wed Mar 21 2007 - wstephenson@suse.de - Fix #200177 - kontact/kmail crash on startup [#208413] - kmail crash in slotGetNextMessage() ++++++ 3_5_BRANCH-r645363.diff ++++++ Index: kmail/searchjob.cpp =================================================================== --- kmail/searchjob.cpp (revision 645362) +++ kmail/searchjob.cpp (revision 645363) @@ -222,7 +222,12 @@ void SearchJob::slotSearchFolder() for ( QStringList::Iterator it = mImapSearchHits.begin(); it != mImapSearchHits.end(); ++it ) { - serNums.append( mFolder->serNumForUID( (*it).toULong() ) ); + ulong serNum = mFolder->serNumForUID( (*it).toULong() ); + // we need to check that the local folder does contain a message for this UID. + // scenario: server responds with a list of UIDs. While the search was running, filtering or bad juju moved a message locally + // serNumForUID will happily return 0 for the missing message, and KMFolderSearch::addSerNum() will fail its assertion. + if ( serNum != 0 ) + serNums.append( serNum ); } emit searchDone( serNums, mSearchPattern, true ); } else { ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Remember to have fun... --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-commit+help@opensuse.org