[Raw Msg Headers][Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Bug in MX record handling...

zmailer/transport/smtp/smtp.c has a problem in its MX-record handling. The
loop, as currently written, will never perform more than one iteration. 

in smtpconn:
for (i = 0; mxh[i].host != NULL; ++i)
	...MX stuff...
	(void) free (mx[i].host);

The result is that, when the test is performed after the first iteration,
mxh[i] is, indeed, NULL, BEFORE 'i' is incremented. I never GETS
incremented and the loop terminates prematurely if there are multiple MXen.

There are a couple of ways this could be rewritten...I was wondering if
any of the more experienced folks out there might have any specific