Subject: Re: ssmtp -t alters To: header of outgoing mail
From: Robert R Schneck <[email-masked]>
To: [email-masked]
Cc: [email-masked]
Date: Fri, 28 Nov 2003 21:37:34 +0000

> Patches gratefully accepted.  Anybody here wanting to take over ssmtp
> maintainership?

A patch is below, I should have included this with my first message.

It occurs to me that this is a bug in ssmtp itself, not just
the Cygwin port.
Who to contact about that, or are you, Corinna, also the overall ssmtp
maintainer?

Sometime in the next few months I might find time to fix up a couple of
other small ssmtp problems.

Robert

*** ../ssmtp.c	Fri Nov 28 21:12:13 2003
--- ssmtp.c	Fri Oct 10 23:41:09 2003
***************
*** 335,346 ****
--- 335,351 ----
  void
  recordRecipient (char *line)
  {
+   int sawSomething = 0;
+
    char buf[MAXLINE];
    char *newaddr;
    char *p = line;

    while ( *p )
      {
+       if (!(isspace(*p))) {
+ 	sawSomething = 1;
+       }
        if ( *p == '"' )
          {
  	  quotes( &p );
***************
*** 357,365 ****
--- 362,381 ----
  	    }
  	  *rec++ = newaddr;
  	  line = p+1;
+ 	  *p = ',';  /* added by Schneck 2003-06-18 */
+ 	  sawSomething = 0;
  	}
        p++;
      }
+
+   /* Schneck 2003-10-10: and what if the , is the end of the line?!
+      Some servers will ignore the result, others will error.
+      Things won't work at all with an address split over two lines! */
+
+   if (sawSomething == 0) {
+     return;
+   }
+
    parseaddr( line, buf, MAXLINE );
    newaddr = strdup( buf );

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/