Re: [Dovecot] Chrooting the Mail processes

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.