I was quite surprised to learn that vdelivermail from vpopmail 5.4.13
and 5.4.17 (and probably most earlier releases) does not detect all
mail loops.
The checks are all performed when parsing .qmail files and the like.
In my particular situation, an Exchange server was picking up mail
from my server (via POP), and deciding to re-distribute it to
everyone in the "To" field (including the mailbox it picked mail
from). I ended up seeing messages in this mailbox with the same
Delivered-To line repeated dozens of times.
I've added a simple fix, which is to check the address right at the
start of main(). I've tested it, and it works. I'd really like to
rework the way Delivered-To headers are checked and added*, but don't
have the time to get into it now. For now, here's the patch if
anyone wants to try it out.
--- vdelivermail-orig.c 2006-06-29 12:36:43.000000000 -0700
+++ vdelivermail.c 2006-10-28 08:17:42.000000000 -0700
@@ -111,8 +111,14 @@
*/
int main(int argc, char **argv)
{
+ char loopcheck[255];
+
/* get the arguments to the program and setup things */
get_arguments(argc, argv);
+ snprintf (loopcheck, sizeof(loopcheck), "[EMAIL PROTECTED]", TheUser,
TheDomain);
+ if ( is_looping( loopcheck ) == 1 ) {
+ vexiterr (EXIT_BOUNCE, "mail is looping");
+ }
#ifdef VALIAS
/* process valiases if configured */
--
Tom Collins - [EMAIL PROTECTED]
Vpopmail - virtual domains for qmail: http://vpopmail.sf.net/
QmailAdmin - web interface for Vpopmail: http://qmailadmin.sf.net/
* I think vdelivermail should insert 1, maybe 2 Delivered-To headers
on each email. The first should be the raw email address from qmail-
local. The second should be the catchall address (or non-ext
address) of the mailbox that ultimately receives the email. The
ultimate delivery address would appear in the headers before the
envelope address.
