opendmarc.c: ARC overwrite for quarantines

[dilyanpalauzov]

ARC overwrite shall not happen only when result == DMARC_RESULT_REJECT, setting possibly result = DMARC_RESULT_ACCEPT, but also when result is DMARC_RESULT_QUARANTINE, calling smfi_quarantine() after ARC is evaluated.

[mskucherawy]

I believe this means:

A "pass" from ARC should move the message disposition to "accept" for both "reject" and "quarantine" policies, but currently this is only done for "reject".

Is that correct?

[dilyanpalauzov]

It sounds reasonable. I do not remember why I wrote this a year ago.

[martinbogo]

I'll take a look at it anyway! Putting this on the triage list as a bug.

[KIC-8462852]

I wrote this patch that fixes ARC overwrite for quarantines.

opendmarc-arc-overwrite-for-quarantines-patch.txt