dovecot-1.0: When creating mailboxes, set cur/new/tmp directorie...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-1.0/rev/8c98ecd43a35
changeset: 5397:8c98ecd43a35
user:      Timo Sirainen <tss at iki.fi>
date:      Sun Sep 09 06:26:01 2007 +0300
description:
When creating mailboxes, set cur/new/tmp directories' group to
dovecot-shared file's group.

diffstat:

1 file changed, 10 insertions(+), 3 deletions(-)
src/lib-storage/index/maildir/maildir-storage.c |   13 ++++++++++---

diffs (37 lines):

diff -r cb3679cd91a1 -r 8c98ecd43a35 src/lib-storage/index/maildir/maildir-storage.c
--- a/src/lib-storage/index/maildir/maildir-storage.c	Sun Sep 09 05:29:41 2007 +0300
+++ b/src/lib-storage/index/maildir/maildir-storage.c	Sun Sep 09 06:26:01 2007 +0300
@@ -34,6 +34,9 @@ struct rename_context {
 
 extern struct mail_storage maildir_storage;
 extern struct mailbox maildir_mailbox;
+
+static const char *maildir_subdirs[] = { "cur", "new", "tmp" };
+#define N_MAILDIR_SUBDIRS (sizeof(maildir_subdirs)/sizeof(maildir_subdirs[0]))
 
 static int verify_inbox(struct maildir_storage *storage,
 			enum mailbox_open_flags *flags);
@@ -643,6 +646,7 @@ static int maildir_create_shared(struct 
 {
 	const char *path;
 	mode_t old_mask;
+	unsigned int i;
 	int fd;
 
 	/* add the execute bit if either read or write bit is set */
@@ -659,9 +663,12 @@ static int maildir_create_shared(struct 
 		umask(old_mask);
 		return -1;
 	}
-	if (chown(dir, (uid_t)-1, gid) < 0) {
-		mail_storage_set_critical(&storage->storage,
-					  "chown(%s) failed: %m", dir);
+	for (i = 0; i < N_MAILDIR_SUBDIRS; i++) {
+		path = t_strconcat(dir, "/", maildir_subdirs[i], NULL);
+		if (chown(path, (uid_t)-1, gid) < 0) {
+			mail_storage_set_critical(&storage->storage,
+						  "chown(%s) failed: %m", path);
+		}
 	}
 
 	path = t_strconcat(dir, "/dovecot-shared", NULL);