arcf_config_free(): don't assert conf->conf_refcnt == 0 #147

glts · 2021-07-25T10:40:59Z

diff --git a/openarc/openarc.c b/openarc/openarc.c
--- a/openarc/openarc.c
+++ b/openarc/openarc.c
@@ -1326,7 +1326,6 @@ static void
 arcf_config_free(struct arcf_config *conf)
 {
        assert(conf != NULL);
-       assert(conf->conf_refcnt == 0);
 
        if (conf->conf_libopenarc != NULL)
                arc_close(conf->conf_libopenarc);

Sendmail does not guarantee that xxfi_close() is called before smfi_main() returns when SIGHUP, SIGTERM or SIGINT is being handled, so this reference counting is incorrect. trusteddomainproject/OpenARC#147

glts changed the title ~~arcf_config_free(): don't assert conf->refcnt == 0~~ arcf_config_free(): don't assert conf->conf_refcnt == 0 Jul 25, 2021

abeverley added a commit to simplelists/OpenARC that referenced this issue Feb 17, 2024

Bug: trusteddomainproject#147

f266b4a

flowerysong mentioned this issue Oct 16, 2024

TDP issues and PRs flowerysong/OpenARC#9

Closed

60 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

arcf_config_free(): don't assert conf->conf_refcnt == 0 #147

arcf_config_free(): don't assert conf->conf_refcnt == 0 #147

glts commented Jul 25, 2021

arcf_config_free(): don't assert conf->conf_refcnt == 0 #147

arcf_config_free(): don't assert conf->conf_refcnt == 0 #147

Comments

glts commented Jul 25, 2021