search for: maildirsize_recalculate

The maildir quota plugin doesn't appear to be including the INBOX in the quota calculation that is reported by GETQUOTAROOT. I took a look at the code and it appears that in this file "src/plugins/quota/quota-maildir.c", the function "maildirsize_recalculate" relies on all of the folders being listed when "maildir_list_init" is called. However the INBOX is not included in that listing, so its current storage size is not included. I couldn't figure out what patch would be needed to fix this, but I'm sure it is something simple....

...0, Bill Boebel wrote: >The maildir quota plugin doesn't appear to be including the INBOX in the quota > calculation that is reported by GETQUOTAROOT. > >I took a look at the code and it appears that in this file > "src/plugins/quota/quota-maildir.c", the function "maildirsize_recalculate" relies > on all of the folders being listed when "maildir_list_init" is called. However the > INBOX is not included in that listing, so its current storage size is not > included. > >I couldn't figure out what patch would be needed to fix this, but I'm sure i...

...sh -> maildirsize_read While reading maildirsize's data, may be situation when mailbox size must be recalculated (maildirsize greater 5120, quotas was changed...). In this case dovecot closes maildirsize's file descriptor (root->fd) and assigns it "-1". After that code calls maildirsize_recalculate -> maildirsize_write So maildirsize is rewritten and contains valid data. BUT file descriptor (root->fd) still contains "-1" 3. User retrieves and deletes some messages, then quits. So maildirsize must be updated. 4. Code calls maildir_quota_transaction_commit, but root->fd equal...