dovecot-2.0: Reverted previous commit. It's a stupid feature: ma...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.0/rev/a8397390075d
changeset: 12019:a8397390075d
user:      Timo Sirainen <tss at iki.fi>
date:      Fri Aug 20 18:22:39 2010 +0100
description:
Reverted previous commit. It's a stupid feature: mail_access_groups already exists.

diffstat:

 src/imap/imap-settings.c     |   2 +-
 src/lmtp/lmtp-settings.c     |   2 +-
 src/master/master-settings.c |  11 -----------
 src/master/master-settings.h |   1 -
 src/pop3/pop3-settings.c     |   2 +-
 5 files changed, 3 insertions(+), 15 deletions(-)

diffs (89 lines):

diff -r 2735d9afd842 -r a8397390075d src/imap/imap-settings.c
--- a/src/imap/imap-settings.c	Fri Aug 20 18:19:50 2010 +0100
+++ b/src/imap/imap-settings.c	Fri Aug 20 18:22:39 2010 +0100
@@ -34,7 +34,7 @@
 	.user = "",
 	.group = "",
 	.privileged_group = "",
-	.extra_groups = "$default_mail_extra_groups",
+	.extra_groups = "",
 	.chroot = "",
 
 	.drop_priv_before_exec = FALSE,
diff -r 2735d9afd842 -r a8397390075d src/lmtp/lmtp-settings.c
--- a/src/lmtp/lmtp-settings.c	Fri Aug 20 18:19:50 2010 +0100
+++ b/src/lmtp/lmtp-settings.c	Fri Aug 20 18:22:39 2010 +0100
@@ -33,7 +33,7 @@
 	.user = "",
 	.group = "",
 	.privileged_group = "",
-	.extra_groups = "$default_mail_extra_groups",
+	.extra_groups = "",
 	.chroot = "",
 
 	.drop_priv_before_exec = FALSE,
diff -r 2735d9afd842 -r a8397390075d src/master/master-settings.c
--- a/src/master/master-settings.c	Fri Aug 20 18:19:50 2010 +0100
+++ b/src/master/master-settings.c	Fri Aug 20 18:22:39 2010 +0100
@@ -175,7 +175,6 @@
 	DEF(SET_ENUM, ssl),
 	DEF(SET_STR, default_internal_user),
 	DEF(SET_STR, default_login_user),
-	DEF(SET_STR, default_mail_extra_groups),
 	DEF(SET_UINT, default_process_limit),
 	DEF(SET_UINT, default_client_limit),
 	DEF(SET_UINT, default_idle_kill),
@@ -203,7 +202,6 @@
 	.ssl = "yes:no:required",
 	.default_internal_user = "dovecot",
 	.default_login_user = "dovenull",
-	.default_mail_extra_groups = "",
 	.default_process_limit = 100,
 	.default_client_limit = 1000,
 	.default_idle_kill = 60,
@@ -259,14 +257,6 @@
 }
 
 static void
-expand_group(const char **group, const struct master_settings *set)
-{
-	/* same reason as expand_user() */
-	if (strcmp(*group, "$default_mail_extra_groups") == 0)
-		*group = set->default_mail_extra_groups;
-}
-
-static void
 fix_file_listener_paths(ARRAY_TYPE(file_listener_settings) *l,
 			pool_t pool, const struct master_settings *master_set,
 			ARRAY_TYPE(const_string) *all_listeners)
@@ -419,7 +409,6 @@
 			}
 		}
 		expand_user(&service->user, &service->user_default, set);
-		expand_group(&service->extra_groups, set);
 		service_set_login_dump_core(service);
 	}
 	set->protocols_split = p_strsplit_spaces(pool, set->protocols, " ");
diff -r 2735d9afd842 -r a8397390075d src/master/master-settings.h
--- a/src/master/master-settings.h	Fri Aug 20 18:19:50 2010 +0100
+++ b/src/master/master-settings.h	Fri Aug 20 18:22:39 2010 +0100
@@ -11,7 +11,6 @@
 	const char *ssl;
 	const char *default_internal_user;
 	const char *default_login_user;
-	const char *default_mail_extra_groups;
 	unsigned int default_process_limit;
 	unsigned int default_client_limit;
 	unsigned int default_idle_kill;
diff -r 2735d9afd842 -r a8397390075d src/pop3/pop3-settings.c
--- a/src/pop3/pop3-settings.c	Fri Aug 20 18:19:50 2010 +0100
+++ b/src/pop3/pop3-settings.c	Fri Aug 20 18:22:39 2010 +0100
@@ -34,7 +34,7 @@
 	.user = "",
 	.group = "",
 	.privileged_group = "",
-	.extra_groups = "$default_mail_extra_groups",
+	.extra_groups = "",
 	.chroot = "",
 
 	.drop_priv_before_exec = FALSE,