[Dovecot] [patch] make <chroot>/./<home> a config option.

[Diego Liziero]

On Fri, 2008-02-15 at 14:53 +0200, Timo Sirainen wrote:
> On Fri, 2008-02-15 at 13:40 +0100, Diego Liziero wrote:
> > This patch adds the bool option home_slash_dot_slash_chroot (feel free
> > to change this name to something easier to understand). Setting this to
> > "no" disables the wu-ftp style /./ chroot.
> 
> There are already too many options, but I guess valid_chroot_dirs could
> be used for this. Committed to v1.1:
> http://hg.dovecot.org/dovecot-1.1/rev/17c65dfdac2a

Great, but this patch solves partially what we would like to have:
it allows chroot options to be completely disabled, but it doesn't allow
to override /./ chroot with a global mail_chroot option.

This happens because to have mail_chroot config option working, we have
to add its entry in valid_chroot_dirs, too.
This should not be necessary.

In this case validate_chroot should be called before checking for
mail_chroot (see the patch below).

Thank you for your quick answer,
Regards,
Diego Liziero.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dovecot-1.0.10-move-validate_chroot.diff
Type: text/x-patch
Size: 938 bytes
Desc: not available
Url : http://dovecot.org/pipermail/dovecot/attachments/20080216/5e965d46/attachment.bin