Reverted previous commit. It's a stupid feature: mail_access_groups already exists.

diff --git a/src/imap/imap-settings.c b/src/imap/imap-settings.c
index e9d072c24641d731490638a1f9a43ed1263241c6..103f9e9abcac97c07d326ecf87e06da1d939c31d 100644 (file)
--- a/src/imap/imap-settings.c
+++ b/src/imap/imap-settings.c
@@ -34,7 +34,7 @@ struct service_settings imap_service_settings = {
        .user = "",
        .group = "",
        .privileged_group = "",
-       .extra_groups = "$default_mail_extra_groups",
+       .extra_groups = "",
        .chroot = "",
 
        .drop_priv_before_exec = FALSE,

diff --git a/src/lmtp/lmtp-settings.c b/src/lmtp/lmtp-settings.c
index ffc127726d1066c2966517c2a1c637e9d9e506db..e9175933231d67c28896597f0f307a9ef3285ecb 100644 (file)
--- a/src/lmtp/lmtp-settings.c
+++ b/src/lmtp/lmtp-settings.c
@@ -33,7 +33,7 @@ struct service_settings lmtp_login_service_settings = {
        .user = "",
        .group = "",
        .privileged_group = "",
-       .extra_groups = "$default_mail_extra_groups",
+       .extra_groups = "",
        .chroot = "",
 
        .drop_priv_before_exec = FALSE,

diff --git a/src/master/master-settings.c b/src/master/master-settings.c
index f183844ce73652069be1cf64c21953747f978089..be642327c8459dbc78954d0139943acdd690ad2c 100644 (file)
--- a/src/master/master-settings.c
+++ b/src/master/master-settings.c
@@ -175,7 +175,6 @@ static const struct setting_define master_setting_defines[] = {
        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 @@ static const struct master_settings master_default_settings = {
        .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,
@@ -258,14 +256,6 @@ expand_user(const char **user, enum service_user_default *default_r,
        }
 }
 
-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,
@@ -419,7 +409,6 @@ master_settings_verify(void *_set, pool_t pool, const char **error_r)
                        }
                }
                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 --git a/src/master/master-settings.h b/src/master/master-settings.h
index 35a74cf496c310afaa22ac4acdbc0810e19c1c27..04f9d03a7cc9e7c5b323b43439497842124135e8 100644 (file)
--- a/src/master/master-settings.h
+++ b/src/master/master-settings.h
@@ -11,7 +11,6 @@ struct master_settings {
        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 --git a/src/pop3/pop3-settings.c b/src/pop3/pop3-settings.c
index 8c0d906f21fcf54a0cd6e0955d65191fbc3ebc7b..0fa239f6ca5fc40e2d7e4b7ac2e31af0b68b411e 100644 (file)
--- a/src/pop3/pop3-settings.c
+++ b/src/pop3/pop3-settings.c
@@ -34,7 +34,7 @@ struct service_settings pop3_service_settings = {
        .user = "",
        .group = "",
        .privileged_group = "",
-       .extra_groups = "$default_mail_extra_groups",
+       .extra_groups = "",
        .chroot = "",
 
        .drop_priv_before_exec = FALSE,