DELAYED DELETE and very long mailboxes => segfault

[Bron Gondwana]

On Wed, Apr 22, 2009 at 01:58:11PM -0400, Brian Awood wrote:
> 
> On Friday 06 February 2009 @ 12:27, Brian Awood wrote:
> > We have had users that weren't able to delete a mailbox that was
> > close to, or already at, the max name length.  Cyrus generates the
> > new name with the DELETED prefix and time stamp suffix, which ends
> > up being longer than the allowed max and the rename then fails.
> 
> While doing testing of 2.3.14 with our local changes I found that we 
> still aren't able to delete mailboxes that are at or near the 
> MAX_MAILBOX_SIZE limit.  It fails in the mboxname_policycheck() at 
> the length test.  The simple solution I see is for the policy check 
> to just return 0 if mboxname_isdeletedmailbox() returns true.  
> Logically, if a mailbox is being deleted, it already exists and 
> therefore should have passed the policy check when it was created.  
> I'm wondering if anyone knows of problems that would cause which I'm 
> not seeing, if not I'll submit a BZ report and diff with the fix.

Yes, fair enough.  That sounds good to me.  I just had a look back
through the logs... the MAX_MAILBOX_BUFFER code is in 2.3.14, so
the bug probably still exists in current CVS.

A patch would be great!

Thanks,

Bron.