Re: Header Problem.

On Tue, Dec 03, 2002 at 12:28:40PM +0000, Paul Matthews wrote:
> Here's the log entry:
> 
> 2002-12-03 00:33:36 18J10J-000507-00 string too long in header_add: 
> 	X-SA-Exim-Rcpt-To: [email protected], [email protected], 
> 	[email protected], allennicola ...
> 
> A quick hack of the sa-exim.c code (about line 274) to alter the header_add
> bit from this:
> 
> 	header_add(' ', "X-SA-Exim-Rcpt-To: %s\n", rcptlist);
> 
> to this:
> 
> 	if (strlen(rcptlist)<8192)
> 		header_add(' ', "X-SA-Exim-Rcpt-To: %s\n", rcptlist);
> 	else
> 		header_add(' ', "X-SA-Exim-Rcpt-To: WARNING - STRING TOO BIG\n");

Mmmh, indeed, I hadn't thought about that.
I'll add a patch like this one in the next version.
(I  think I'll  add an  option that  doesn't add  the X-SA-Exim-Rcpt-To:
header if  it is bigger  than a configurable  size, giving the  user the
option to set it to 0)

(BTW, you want your patch to say strlen(rcptlist)<8100), you have to
acount for the size of X-SA-Exim-Rcpt-To:, the newline at the end and so
forth :-)

Marc
-- 
"A mouse is a device used to point at the xterm you want to type in" - A.S.R.
Microsoft is to operating systems & security ....
                                      .... what McDonalds is to gourmet cooking 
Home page: http://marc.merlins.org/   |   Finger [email protected] for PGP key