>If it can't read a control file, it should say so, to stderr.
>(e.g. "unable to read rcpthosts")
Just a note that usually it is not an error for a control file to be
missing. The control files mostly only override default settings. For
rcpthosts in particular, my complaint would be that the default
behaviour is IMHO wrong; if there is no rcpthosts, qmail-smtpd should
refuse to relay anything as opposed to relaying everything. If you want
to relay everything, you should have to create a file with a single "#"
in it. But that is a side issue :-)
I think you've got the right idea. The main one for me is that if
qmail-smtpd sends any sort of error to a client, an error (hopefully
slightly more verbose) should also be written to stderr. That should be
pretty trivial to do since I believe all the error output is done in
only a few functions.
Evan
