On Wednesday, October 1, 2003, at 03:20 PM, Jochen Schug wrote:
I tried adding the trailing slash, there's the result:

Please remove the trailing slash and try this patch. The changes in vdelivermail resulted in a double-slash (adding a / and the /Maildir/). The result is that mail is delivered fine, but domain quotas won't work.


------------------------------
diff -u -r1.3 vdelivermail.c
--- vdelivermail.c      14 Sep 2003 21:51:59 -0000      1.3
+++ vdelivermail.c      1 Oct 2003 22:39:53 -0000
@@ -1089,12 +1089,12 @@

/* check if it is a path add the /Maildir/ for delivery */
if ( bounce[0] == '/' ) {
- strcat( bounce, "/");
+ if (bounce[strlen(bounce)-1] != '/') strcat( bounce, "/");
printf ("user does not exist, but will deliver to %s\n", bounce);
if( check_forward_deliver(bounce) == 1 )
vexit(0);
else
- strcat( bounce, "/Maildir/");
+ strcat( bounce, "Maildir/");
}


     ret = deliver_mail(bounce, "NOQUOTA" );
---------------------------

If it's a good fix, I'll include it in the 5.3.28 release (which is planned for later today).

This should fix a few things. It only adds a trailing slash if one isn't there, and it doesn't double-up the slash when adding /Maildir/.

--
Tom Collins
[EMAIL PROTECTED]
QmailAdmin: http://qmailadmin.sf.net/  Vpopmail: http://vpopmail.sf.net/
Info on the Sniffter hand-held Network Tester: http://sniffter.com/




Reply via email to