From 554e333ebbe0552b61604cdf8fec77f507a3ab9b Mon Sep 17 00:00:00 2001 From: Phil Pennock Date: Mon, 21 Feb 2011 22:17:13 -0500 Subject: [PATCH] Don't disable quota when maildirsize lost to races. When maildir_ensure_sizefile() returns -2, we still have size information, so we can still use that. Don't disable quota. As a result, do refrain from potentially calling close(-2). Fixes bug 1086 --- src/src/transports/appendfile.c | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/src/src/transports/appendfile.c b/src/src/transports/appendfile.c index e5ab65765..39dd4f8f0 100644 --- a/src/src/transports/appendfile.c +++ b/src/src/transports/appendfile.c @@ -2419,18 +2419,12 @@ else "%s/maildirsize", check_path); return FALSE; } - else if (maildirsize_fd == -2) - { - DEBUG(D_transport) debug_printf("disabling quota check because of " - "races updating %s/maildirsize", check_path); - disable_quota = TRUE; - } + /* can also return -2, which means that the file was removed because of + raciness; but in this case, the size & filecount will still have been + updated. */ - if (maildirsize_fd >= 0) - { - if (mailbox_size < 0) mailbox_size = size; - if (mailbox_filecount < 0) mailbox_filecount = filecount; - } + if (mailbox_size < 0) mailbox_size = size; + if (mailbox_filecount < 0) mailbox_filecount = filecount; } /* No quota enforcement; ensure file does *not* exist; calculate size if @@ -2927,7 +2921,8 @@ if (!disable_quota) if (yield == OK && maildirsize_fd >= 0) maildir_record_length(maildirsize_fd, message_size); maildir_save_errno = errno; /* Preserve errno while closing the file */ - (void)close(maildirsize_fd); + if (maildirsize_fd >= 0) + (void)close(maildirsize_fd); errno = maildir_save_errno; } #endif /* SUPPORT_MAILDIR */ -- 2.25.1