[Dovecot] Chrooting the Mail processes

[Timo Sirainen]

On Sunday, Aug 24, 2003, at 21:17 Europe/Helsinki, Bert Koelewijn wrote:

> I think you forgot to post some diff's:
>
> MEMBER(valid_chroot_dirs) NULL,
> MEMBER(mail_chroot) ...

Oh, forgot to add it myself too. Looks like gcc 3 doesn't anymore 
complain if some of those are missing.

> and this can't be right:
>
> static int validate_chroot(const char *dir)
> {
>         const char *const *chroot_dirs;
>
>         if (*dir == '\0')
>                 return FALSE;
>
>         if (set->valid_chroot_dirs == NULL)
>                 return FALSE;

I don't see why not? The point of that function is to return TRUE if 
given directory is found from valid_chroot_dirs list.