<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] Add $umask for mailboxes and attachments



On Wednesday, 21 March 2007 at 10:19, Brendan Cully wrote:
> So how about the following strategy for $umask:
> 
> 1. I back out my patch
> 
> 2. we save the existing $umask when setting it to 077 in main
> 3. we temporarily restore that umask for saving attachments and pipe
>    commands
> 4. we create new mailboxes unconditionally as 077, but use the
>    permissions of the existing mailbox when updating it. This works
>    for mbox, but we would need to stat the root directory of
>    maildir/mh and save that mode when updating.

I've implemented step 4, which I think was clearly a bug.
I'm still on the fence about 3.

Attachment: pgpB9oohwgAVp.pgp
Description: PGP signature