2 messages in net.sourceforge.lists.courier-users[courier-users] Webmail patch.
FromSent OnAttachments
Sam VarshavchikDec 17, 2000 9:50 pm 
Daniel BiddleDec 18, 2000 12:11 am 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:[courier-users] Webmail patch.Actions...
From:Sam Varshavchik (mrs@courier-Mta.com)
Date:Dec 17, 2000 9:50:21 pm
List:net.sourceforge.lists.courier-users

Webmail just choked on the last courier-users message. It turned out to be a null pointer, with an easy two-line fix. This bug's been there for a little while. That code is only used to show RFC 2369 List- headers, and the bug occurs when there's more than one address in the same List- header. I guess a new version of mailman got installed, which came with this "feature". Multiple-address List- headers are not very common (this is the first time I've seen it). Go figure.

Apply the following patch to webmail/folder.c. Just in time to be included in the next major release.

-- Sam

Index: sqwebmail/folder.c =================================================================== RCS file: /cvsroot/courier/courier/webmail/folder.c,v retrieving revision 1.67 diff -U3 -r1.67 folder.c --- sqwebmail/folder.c 2000/12/10 03:06:22 1.67 +++ sqwebmail/folder.c 2000/12/18 05:22:50 @@ -1238,8 +1238,11 @@

printf("</A>");

- sai->curindex++; - sai->isfirstchar=1; + if (sai) + { + sai->curindex++; + sai->isfirstchar=1; + }

printf("%s", sep); }