On Mon, Aug 16, 2004 at 04:52:48PM +0100, Philip Hazel wrote: > On Mon, 16 Aug 2004, Andy Wettstein wrote:
>
> >BOOL use_chown = parent == spool_directory && geteuid() == root_uid;
> >...
> >if (use_chown) Uchown(buffer, exim_uid, exim_gid);
> >
> >So I wonder what the purpose of that if statement is because I don't
> >think this would ever really happen because exim won't deliver as
> >root (unless you change the fixed_never_users at compile time).
>
> The function in which that appears is used to create all kinds of
> directories; in particular, for a new installation is is used to create
> the spool directories. This can happen when Exim is running as root; it
> therefore ensures that in that case, they are changed to exim:exim.
Yes, I think a realized that a little bit after I sent that message
(after I thought about the meaning of spool_directory).
So would it be possible to have chown called for the directory during
delivery, too? I know now that I can at least change the parent
directory to get correct group permissions for shared email, but I would
think that the expected behavior would be to set the group ownership
based on the group option.
