Bug 49301

Summary: UIDL pop3 command mishandles commas in the unique identifer
Product: [Unmaintained] kmail Reporter: Andrew Rivett <andrew>
Component: generalAssignee: kdepim bugs <pim-bugs-null>
Status: RESOLVED NOT A BUG    
Severity: normal    
Priority: NOR    
Version First Reported In: 1.4   
Target Milestone: ---   
Platform: FreeBSD Ports   
OS: All   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Andrew Rivett 2002-10-17 21:06:52 UTC 
Version:           1.4 (using KDE KDE 3.0.3)
Installed from:    FreeBSD Ports
OS:          FreeBSD

When the pop3 server returns commas in the UIDL for messages, it is stored in the config file as: \\, which inturns doesn't match the next time around.   I suspect the extra slash is causing the problem when matching the next time the uidl is fetched.  Here is an example.

UIDL
+OK
1 10028399918.107728_1.machine1-12a:2,
.

This is stored in the kmail config file for that pop server as:
downloadLater=
seenUidList=10028399918.107728_1.machine1-12a:2\\,

If you need any other debug information, please let me know.
Comment 1 Ingo Klöcker 2002-10-19 23:03:50 UTC 
Subject: Re:  New: UIDL pop3 command mishandles commas in the unique identifer

On Thursday 17 October 2002 21:06, Andrew Rivett wrote:
> This is stored in the kmail config file for that pop server as:
> downloadLater=
> seenUidList=10028399918.107728_1.machine1-12a:2\\,
>
> If you need any other debug information, please let me know.

I just found a bug in the KDE libraries which causes the last entry in 
the seenUidList not to be loaded if it ends with a comma. Please add 
the following code to KMAcctExpPop::processNewMail(...) and watch the 
debug output. I'd like to know whether only the last UID is missing or 
whether there is something else wrong.
=====
--- kmacctexppop.cpp~   Fri Oct 11 23:28:45 2002
+++ kmacctexppop.cpp    Sat Oct 19 22:07:40 2002
@@ -157,6 +157,11 @@ void KMAcctExpPop::processNewMail(bool _
     headerLaterUids = config.readListEntry( "downloadLater" );
     uidsOfNextSeenMsgs.clear();

+for ( QStringList::Iterator it = uidsOfSeenMsgs.begin();
+      it != uidsOfSeenMsgs.end(); ++it ) {
+  kdDebug(5006) << "ooooo seenUid: '" << *it << "'" << endl;
+}
+
     interactive = _interactive;
     mUidlFinished = FALSE;
     startJob();
=====

Regards,
Ingo
Comment 2 Stephan Kulow 2003-09-24 12:10:52 UTC 
no response for 11 months