Cyrus 3.0.4 - Over quota returns 550-Mailbox unknown
Hello, i'm using compiled cyrus version 3.0.4 on Debian 9.2. When the Postfix server is trying to deliver a message via lmtp to cyrus mailbox which is over quota, it receives the 550-Mailbox unknown return code and not the 452 or 552 Over quota. In the previous version 2.5.10 it worked fine with default values in imapd.conf Is it a bug or my fault? Thanks Maros Vegh Cyrus Home Page: http://www.cyrusimap.org/ List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ To Unsubscribe: https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
Re: Cyrus 3.0.4 - Over quota returns 550-Mailbox unknown
Hi, that's surprising and the same here. I think there is a bug in imap/lmtpengine.c, function process_recipient: verify_user returns the correct error code as we can see with LOG_DEBUG "on": Nov 2 14:10:50 popc lmtp[2092]: verify_user(t...@sendmaid.org) failed: Over quota but this return code is not used in process_recipient There we have: [...] if (sl) { char *rcpt = xstrndup(addr, sl); mbname = mbname_from_recipient(rcpt, msg->ns); free(rcpt); int forcedowncase = config_getswitch(IMAPOPT_LMTP_DOWNCASE_RCPT); if (forcedowncase) mbname_downcaseuser(mbname); /* strip username if postuser */ if (!strcmpsafe(mbname_localpart(mbname), config_getstring(IMAPOPT_POSTUSER))) { mbname_set_localpart(mbname, NULL); if (!config_virtdomains || !strcmpsafe(mbname_domain(mbname), config_defdomain)) mbname_set_domain(mbname, NULL); } if (verify_user(mbname, (quota_t) (ignorequota ? -1 : msg->size), ignorequota ? -1 : 1, msg->authstate)) { mbname_free(&mbname); } } if (!mbname) { const char *catchall = config_getstring(IMAPOPT_LMTP_CATCHALL_MAILBOX); if (catchall) { mbname = mbname_from_userid(catchall); if (verify_user(mbname, ignorequota ? -1 : msg->size, ignorequota ? -1 : 1, msg->authstate)) { mbname_free(&mbname); } } } if (!mbname) { /* we lost */ return IMAP_MAILBOX_NONEXISTENT; } [...] means as far as I understand: if verify_user returns its error (for example IMAP_QUOTA_EXCEEDED), mbname is freed and process_recipient always returns IMAP_MAILBOX_NONEXISTENT. Below is a patch that works for me. But I don't know if this is a good way to fix it. Hopefully one of the developers helps :) Regards, Edda diff -Naur cyrus-imapd-3.0.4.orig/imap/lmtpengine.c cyrus-imapd-3.0.4/imap/lmtpengine.c --- cyrus-imapd-3.0.4.orig/imap/lmtpengine.c 2017-09-04 02:09:46.0 +0200 +++ cyrus-imapd-3.0.4/imap/lmtpengine.c 2017-11-02 13:59:56.764175245 +0100 @@ -830,6 +830,7 @@ } mbname_t *mbname = NULL; + int r = 0; size_t sl = strlen(addr); if (addr[sl-1] == '>') sl--; @@ -849,7 +850,7 @@ mbname_set_domain(mbname, NULL); } - if (verify_user(mbname, + if (r = verify_user(mbname, (quota_t) (ignorequota ? -1 : msg->size), ignorequota ? -1 : 1, msg->authstate)) { mbname_free(&mbname); @@ -860,7 +861,7 @@ const char *catchall = config_getstring(IMAPOPT_LMTP_CATCHALL_MAILBOX); if (catchall) { mbname = mbname_from_userid(catchall); - if (verify_user(mbname, + if (r = verify_user(mbname, ignorequota ? -1 : msg->size, ignorequota ? -1 : 1, msg->authstate)) { mbname_free(&mbname); @@ -870,6 +871,9 @@ if (!mbname) { /* we lost */ + if (r) { + return r; + } return IMAP_MAILBOX_NONEXISTENT; } Am 02.11.17 um 09:27 schrieb Maros Vegh: Hello, i'm using compiled cyrus version 3.0.4 on Debian 9.2. When the Postfix server is trying to deliver a message via lmtp to cyrus mailbox which is over quota, it receives the 550-Mailbox unknown return code and not the 452 or 552 Over quota. In the previous version 2.5.10 it worked fine with default values in imapd.conf Is it a bug or my fault? Thanks Maros Vegh Cyrus Home Page: http://www.cyrusimap.org/ List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ To Unsubscribe: https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
Re: Cyrus 3.0.4 - Over quota returns 550-Mailbox unknown
I have created https://github.com/cyrusimap/cyrus-imapd/issues/2191 to track this. Thanks for the patch! Cheers, Bron. On Fri, 3 Nov 2017, at 00:38, Edda wrote: > Hi, > > that's surprising and the same here. > > I think there is a bug in imap/lmtpengine.c, function > process_recipient:> > verify_user returns the correct error code as we can see with > LOG_DEBUG> "on": > > Nov 2 14:10:50 popc lmtp[2092]: verify_user(t...@sendmaid.org) > failed:> Over quota > > but this return code is not used in process_recipient > > There we have: > [...] > if (sl) { > char *rcpt = xstrndup(addr, sl); > mbname = mbname_from_recipient(rcpt, msg->ns); > free(rcpt); > > int forcedowncase = > config_getswitch(IMAPOPT_LMTP_DOWNCASE_RCPT);> if > (forcedowncase) mbname_downcaseuser(mbname); > > /* strip username if postuser */ > if (!strcmpsafe(mbname_localpart(mbname), > config_getstring(IMAPOPT_POSTUSER))) { > mbname_set_localpart(mbname, NULL); > if (!config_virtdomains || > !strcmpsafe(mbname_domain(mbname), config_defdomain)) > mbname_set_domain(mbname, NULL); > } > > if (verify_user(mbname, > (quota_t) (ignorequota ? -1 : msg->size), > ignorequota ? -1 : 1, msg->authstate)) { > mbname_free(&mbname); > } > } > > if (!mbname) { > const char *catchall = > config_getstring(IMAPOPT_LMTP_CATCHALL_MAILBOX); > if (catchall) { > mbname = mbname_from_userid(catchall); > if (verify_user(mbname, > ignorequota ? -1 : msg->size, > ignorequota ? -1 : 1, msg->authstate)) { > mbname_free(&mbname); > } > } > } > > if (!mbname) { > /* we lost */ > return IMAP_MAILBOX_NONEXISTENT; > } > [...] > > means as far as I understand: if verify_user returns its error (for > example IMAP_QUOTA_EXCEEDED), mbname is freed and process_recipient > always returns IMAP_MAILBOX_NONEXISTENT. > > Below is a patch that works for me. But I don't know if this is a good> way > to fix it. > Hopefully one of the developers helps :) > > Regards, Edda > > > diff -Naur cyrus-imapd-3.0.4.orig/imap/lmtpengine.c > cyrus-imapd-3.0.4/imap/lmtpengine.c > --- cyrus-imapd-3.0.4.orig/imap/lmtpengine.c2017-09-04 > 02:09:46.0 +0200 > +++ cyrus-imapd-3.0.4/imap/lmtpengine.c2017-11-02 > 13:59:56.764175245> +0100 > @@ -830,6 +830,7 @@ > } > > mbname_t *mbname = NULL; > +int r = 0; > > size_t sl = strlen(addr); > if (addr[sl-1] == '>') sl--; > @@ -849,7 +850,7 @@ > mbname_set_domain(mbname, NULL); > } > > -if (verify_user(mbname, > +if (r = verify_user(mbname, > (quota_t) (ignorequota ? -1 : msg->size), > ignorequota ? -1 : 1, msg->authstate)) { > mbname_free(&mbname); > @@ -860,7 +861,7 @@ > const char *catchall = > config_getstring(IMAPOPT_LMTP_CATCHALL_MAILBOX); > if (catchall) { > mbname = mbname_from_userid(catchall); > -if (verify_user(mbname, > +if (r = verify_user(mbname, > ignorequota ? -1 : msg->size, > ignorequota ? -1 : 1, msg->authstate)) {> > mbname_free(&mbname); > @@ -870,6 +871,9 @@ > > if (!mbname) { > /* we lost */ > +if (r) { > +return r; > +} > return IMAP_MAILBOX_NONEXISTENT; > } > > > > Am 02.11.17 um 09:27 schrieb Maros Vegh: >> Hello, >> >> i'm using compiled cyrus version 3.0.4 on Debian 9.2. >> >> When the Postfix server is trying to deliver a message via lmtp to >> cyrus mailbox which is over quota, it receives the 550-Mailbox >> unknown>> return code and not the 452 or 552 Over quota. >> >> In the previous version 2.5.10 it worked fine with default values in>> >> imapd.conf >> >> Is it a bug or my fault? >> >> Thanks >> >> Maros Vegh >> >> > > > Cyrus Home Page: http://www.cyrusimap.org/ > List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/> To > Unsubscribe: > https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus -- Bron Gondwana, CEO, FastMail Pty Ltd br...@fastmailteam.com Cyrus Home Page: http://www.cyrusimap.org/ List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/ To Unsubscribe: https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus