search for: mail_error_temp

Hi, Hopefully this is the right channel for such a patch. I have a minor enhancement to submit for the antispam plugin http://hg.dovecot.org/dovecot-antispam-plugin It adds minimal error checking for the sendmail_binary, otherwise the reported error in case of a missing binary or one with missing permissions is generic and not useful. Thanks, Robert -- http://robert.muntea.nu/

On 10/18/20 10:28 PM, Aki Tuomi wrote: >> doveadm(myuser at example.com): Panic: file mail-storage.c: line 2112 (mailbox_get_open_status): assertion failed: (box->opened) ... > I can reproduce your problem with the `fts lookup` command. Luckily it's equivalent to running `doveadm search`. I'll open a bug about this. Can you provide any status on the bug/fix? Thanks.

...box_alloc(info->ns->list, info->vname, 0); +??? mailbox_set_reason(box,"fts search"); +??? if (mailbox_open(box) < 0) { +??? ??? i_error("Couldn't open mailbox: %s", +??? ??? ??? mailbox_get_last_internal_error(box, NULL)); +??? ??? doveadm_mail_failed_error(ctx, MAIL_ERROR_TEMP); +??? ??? return -1; +??? } + ???? if (fts_backend_lookup(backend, box, ctx->search_args->args, ???? ??? ??? ??? ????? FTS_LOOKUP_FLAG_AND_ARGS, &result) < 0) { ???? ??? i_error("fts lookup failed"); On a more minor issue, with this patch if you search for a non existent ma...

...gt;> >> Thanks, >> >> Robert > > Robert, I like that you did this. > > Beyond that and without even looking at the actual code, I'm curious why > you: > > + if (access(cfg->binary, F_OK) == -1) > + { > + mail_storage_set_error(storage, MAIL_ERROR_TEMP, "mail_sendmail > file does not exist"); > > instead of finding a way to include the value of cfg->binary in the > error message string. > > This might not be needed if it's really obvious from the config file > what the path to the executable is, but if there...

...0x55945fe73ee0 "admin at usrpro.io" > ??????? box_vname = 0x55945fe7f310 "Sent" > ??????? errstr = 0x7f165469cb53 <quota_mailbox_sync_deinit+164> > "\213E\344H\203\304([]\351\320f\377\377UH\211\345H\203\354\060H\211}\330dH\213\004%(" > ??????? error = MAIL_ERROR_TEMP > ??????? status = {messages = 10, recent = 0, unseen = 0, uidvalidity = > 1545140227, uidnext = 11, first_unseen_seq = 0, first_recent_uid = 3, > last_cached_seq = 1, highest_modseq = 0, highest_pvt_modseq = 0, > keywords = 0x0, permanent_flags = 0, flags = 0, permanent_keywords = >...

...rpro.io" >>> ??????? box_vname = 0x55945fe7f310 "Sent" >>> ??????? errstr = 0x7f165469cb53 <quota_mailbox_sync_deinit+164> >>> "\213E\344H\203\304([]\351\320f\377\377UH\211\345H\203\354\060H\211}\330dH\213\004%(" >>> ??????? error = MAIL_ERROR_TEMP >>> ??????? status = {messages = 10, recent = 0, unseen = 0, uidvalidity = >>> 1545140227, uidnext = 11, first_unseen_seq = 0, first_recent_uid = 3, >>> last_cached_seq = 1, highest_modseq = 0, highest_pvt_modseq = 0, >>> keywords = 0x0, permanent_flags = 0, flag...

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hello, I've put together an always tempfail & sendmail-like DSN report back patch. Because I don't know how to transform the rejection reason into sysexit.h codes, it always tempfails. The hooks are so that: a) config file setting: lda_mode = mode b) command line: deliver --mode mode where mode is default or sendmail. When no

...??????? username = 0x55945fe73ee0 "admin at usrpro.io" ??????? box_vname = 0x55945fe7f310 "Sent" ??????? errstr = 0x7f165469cb53 <quota_mailbox_sync_deinit+164> "\213E\344H\203\304([]\351\320f\377\377UH\211\345H\203\354\060H\211}\330dH\213\004%(" ??????? error = MAIL_ERROR_TEMP ??????? status = {messages = 10, recent = 0, unseen = 0, uidvalidity = 1545140227, uidnext = 11, first_unseen_seq = 0, first_recent_uid = 3, last_cached_seq = 1, highest_modseq = 0, highest_pvt_modseq = 0, keywords = 0x0, permanent_flags = 0, flags = 0, permanent_keywords = false, allow_new_keyword...

...quot;admin at usrpro.io" >> ??????? box_vname = 0x55945fe7f310 "Sent" >> ??????? errstr = 0x7f165469cb53 <quota_mailbox_sync_deinit+164> >> "\213E\344H\203\304([]\351\320f\377\377UH\211\345H\203\354\060H\211}\330dH\213\004%(" >> ??????? error = MAIL_ERROR_TEMP >> ??????? status = {messages = 10, recent = 0, unseen = 0, uidvalidity = >> 1545140227, uidnext = 11, first_unseen_seq = 0, first_recent_uid = 3, >> last_cached_seq = 1, highest_modseq = 0, highest_pvt_modseq = 0, >> keywords = 0x0, permanent_flags = 0, flags = 0, permanent...

...t;>>> ??????? box_vname = 0x55945fe7f310 "Sent" >>>> ??????? errstr = 0x7f165469cb53 <quota_mailbox_sync_deinit+164> >>>> "\213E\344H\203\304([]\351\320f\377\377UH\211\345H\203\354\060H\211}\330dH\213\004%(" >>>> ??????? error = MAIL_ERROR_TEMP >>>> ??????? status = {messages = 10, recent = 0, unseen = 0, uidvalidity = >>>> 1545140227, uidnext = 11, first_unseen_seq = 0, first_recent_uid = 3, >>>> last_cached_seq = 1, highest_modseq = 0, highest_pvt_modseq = 0, >>>> keywords = 0x0, permanent...

...8 = 0}, metadata = {guid = "\027\000\000\000\000\000\000\000\000\337\354\000\027\225\023J", virtual_size = 355707281760, cache_fields = 0x52d1cd1470, precache_fields = 3519963648, backend_ns_prefix = 0x52d1ce6110 "", backend_ns_type = 3519963488}, error = MAIL_ERROR_TEMP, errstr = 0x52d1ca7288 "\240\200\312\321R"} ns = 0x52d1ce1ab0 mailbox = 0x52d1c9e160 "virtual/Expunged" orig_mailbox = 0x52d1ce8ac8 "virtual/Expunged" selected_mailbox = false #9 0x00000052d019aebc in command_exec (cmd=cmd at entry=0x52...

Hope this helps. There are no debugging symbols available for dovecot in the alpine repository. (gdb) bt full #0? a_crash () at ./arch/x86_64/atomic_arch.h:108 No locals. #1? free (p=0x55d949250660) at src/malloc/malloc.c:467 ??????? extra = 65 ??????? base = 0x55d94925060f "" ??????? len = 94391723427649 ??????? self = 0x55d949250650 ??????? next = <optimized out> ???????