dovecot-1.1: Mailbox renaming with LAYOUT=fs wasn't renaming con...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-1.1/rev/24f9aca75de0
changeset: 8285:24f9aca75de0
user:      Timo Sirainen <tss at iki.fi>
date:      Fri May 22 13:07:17 2009 -0400
description:
Mailbox renaming with LAYOUT=fs wasn't renaming control dirs if they were used.

diffstat:

1 file changed, 24 insertions(+), 15 deletions(-)
src/lib-storage/list/mailbox-list-fs.c |   39 +++++++++++++++++++-------------

diffs (59 lines):

diff -r 255af68d5957 -r 24f9aca75de0 src/lib-storage/list/mailbox-list-fs.c
--- a/src/lib-storage/list/mailbox-list-fs.c	Thu May 21 12:45:50 2009 -0400
+++ b/src/lib-storage/list/mailbox-list-fs.c	Fri May 22 13:07:17 2009 -0400
@@ -286,10 +286,30 @@ static int fs_list_delete_mailbox(struct
 	return mailbox_list_delete_index_control(list, name);
 }
 
+static int rename_dir(struct mailbox_list *list,
+		      enum mailbox_list_path_type type,
+		      const char *oldname, const char *newname)
+{
+	const char *oldpath, *newpath;
+
+	oldpath = mailbox_list_get_path(list, oldname, type);
+	newpath = mailbox_list_get_path(list, newname, type);
+
+	if (strcmp(oldpath, newpath) == 0)
+		return 0;
+
+	if (rename(oldpath, newpath) < 0 && errno != ENOENT) {
+		mailbox_list_set_critical(list, "rename(%s, %s) failed: %m",
+					  oldpath, newpath);
+		return -1;
+	}
+	return 0;
+}
+
 static int fs_list_rename_mailbox(struct mailbox_list *list,
 				  const char *oldname, const char *newname)
 {
-	const char *oldpath, *newpath, *old_indexdir, *new_indexdir, *p;
+	const char *oldpath, *newpath, *p;
 	struct stat st;
 
 	oldpath = mailbox_list_get_path(list, oldname,
@@ -347,20 +367,9 @@ static int fs_list_rename_mailbox(struct
 		return -1;
 	}
 
-	/* we need to rename the index directory as well */
-	old_indexdir = mailbox_list_get_path(list, oldname,
-					     MAILBOX_LIST_PATH_TYPE_INDEX);
-	new_indexdir = mailbox_list_get_path(list, newname,
-					     MAILBOX_LIST_PATH_TYPE_INDEX);
-	if (*old_indexdir != '\0') {
-		if (rename(old_indexdir, new_indexdir) < 0 &&
-		    errno != ENOENT) {
-			mailbox_list_set_critical(list,
-						  "rename(%s, %s) failed: %m",
-						  old_indexdir, new_indexdir);
-		}
-	}
-
+	(void)rename_dir(list, MAILBOX_LIST_PATH_TYPE_CONTROL,
+			 oldname, newname);
+	(void)rename_dir(list, MAILBOX_LIST_PATH_TYPE_INDEX, oldname, newname);
 	return 0;
 }