wiz> Synopsis: spamass-milter dies with a NULL body spam mail
wiz> 
wiz> State-Changed-From-To: open->feedback
wiz> State-Changed-By: wiz@netbsd.org
wiz> State-Changed-When: Sat, 19 Mar 2005 14:35:36 +0000
wiz> State-Changed-Why:
wiz> I just updated the package to 0.3.0.
wiz> 1. is fixed by it; there's a -f that should fix 2.
wiz> I'm not sure if 3. is fixed. Could you please check?
wiz> Thanks.

Thank you for your mail.

As you mentioned, two problems are fixed by 0.3.0.
But I think we still need some hack.
My changes are:

   Always use macro_b, so SA can do future/past validation.
   It is not a bug, but it makes SA happier.

   SA 2.x requires "($v/$Z)$?r with $r" part of Received: header.
   I don't think SA 3.x need this, but keep it for SA 2.x users.

   There is a bug in cmp_nocase_partial() function. Without my hack,
   cmp_nocase_partial("X-Spam-", "X-Spam") returns 0 which is
   incorrect behavior for this function.

Hope this helps you.

Yoshitaka Tokugawa

--- spamass-milter.cpp.orig	Sun Mar 20 10:10:25 2005
+++ spamass-milter.cpp	Sun Mar 20 10:32:22 2005
@@ -898,13 +898,21 @@
 				(envelope-from $g)$.
 		   
 		*/
-		const char *macro_b, *macro_s, *macro_j, *macro__;
+		const char *macro_b, *macro_s, *macro_j, *macro__, *macro_r;
+		char date[32];	/* sizeof("Mon, 23 Aug 2004 18:28:43 +0900") */
+		time_t tval;
 
 		/* Failure to fetch {b} is not fatal.  Without this date SA can't do
 		   future/past validation on the Date: header, but sendmail doesn't
 		   default to allow milters to see it.
 		*/
 		macro_b = smfi_getsymval(ctx, "b");
+ 		if (!macro_b)
+		{
+			(void) time(&tval);
+			(void) strftime(date, sizeof(date), "%a, %d %b %Y %H:%M:%S %z", localtime(&tval));
+			macro_b = date;
+		}
 			
 		/* Sendmail currently cannot pass us the {s} macro, but
 		   I do not know why.  Leave this in for the day sendmail is
@@ -916,6 +924,10 @@
 			macro_s = sctx->helo;
 		if (!macro_s)
 			macro_s = "nohelo";
+		/* Get mailer info. */
+		macro_r = smfi_getsymval(ctx, "r");
+		if (!macro_r)
+			macro_r = "SMTP"; /* Assuming it as SMTP */
 
 		/* FQDN of this site */
 		macro_j = smfi_getsymval(ctx, "j");
@@ -930,10 +942,10 @@
 		assassin->output((string)"X-Envelope-From: "+assassin->from()+"\r\n");
 		assassin->output((string)"X-Envelope-To: "+envrcpt[0]+"\r\n");
 
-		if (!macro_b)
-			assassin->output((string)"Received: from "+macro_s+" ("+macro__+") by "+macro_j+";\r\n");
-		else
-			assassin->output((string)"Received: from "+macro_s+" ("+macro__+") by "+macro_j+"; "+macro_b+"\r\n");
+		assassin->output((string)"Received: from "+macro_s
+			+" ("+macro__+")\n\tby "+macro_j
+			+" ("+PACKAGE_NAME+"/"+PACKAGE_VERSION	/* dummy */
+			+") with "+macro_r+";\n\t"+macro_b+"\r\n");
 
 	} else
 		assassin->output((string)"X-Envelope-To: "+envrcpt[0]+"\r\n");
@@ -1965,7 +1977,7 @@
   string::const_iterator p=s.begin();
   string::const_iterator p2=s2.begin();
 
-  while ( p != s.end() && p2 != s2.end() ) {
+  while ( p != s.end() ) {
     if (toupper(*p) != toupper(*p2))
     {
       debug(D_STR, "c_nc_p: <%s><%s> : miss", s.c_str(), s2.c_str());