dovecot-1.2: master: If login process has -D parameter, make log...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-1.2/rev/71cfd00e1375
changeset: 8821:71cfd00e1375
user:      Timo Sirainen <tss at iki.fi>
date:      Fri Mar 13 15:11:46 2009 -0400
description:
master: If login process has -D parameter, make login directory writable.

diffstat:

1 file changed, 15 insertions(+), 1 deletion(-)
src/master/master-settings.c |   16 +++++++++++++++-

diffs (33 lines):

diff -r 6131143fef16 -r 71cfd00e1375 src/master/master-settings.c
--- a/src/master/master-settings.c	Fri Mar 13 14:50:41 2009 -0400
+++ b/src/master/master-settings.c	Fri Mar 13 15:11:46 2009 -0400
@@ -941,6 +941,19 @@ static bool settings_verify(struct setti
 	return TRUE;
 }
 
+static bool login_want_core_dumps(struct settings *set)
+{
+	const char *p;
+
+	p = strstr(set->server->pop3->login_executable, " -D");
+	if (p != NULL && p[3] == '\0')
+		return TRUE;
+	p = strstr(set->server->imap->login_executable, " -D");
+	if (p != NULL && p[3] == '\0')
+		return TRUE;
+	return FALSE;
+}
+
 static bool settings_do_fixes(struct settings *set)
 {
 	struct stat st;
@@ -972,7 +985,8 @@ static bool settings_do_fixes(struct set
 		   empty. with external auth we wouldn't want to delete
 		   existing sockets or break the permissions required by the
 		   auth server. */
-		if (safe_mkdir(set->login_dir, 0750,
+		mode_t mode = login_want_core_dumps(set) ? 0770 : 0750;
+		if (safe_mkdir(set->login_dir, mode,
 			       master_uid, set->server->login_gid) == 0) {
 			i_warning("Corrected permissions for login directory "
 				  "%s", set->login_dir);