[Dovecot] Released 1.0.rc31

Timo Sirainen tss at iki.fi
Tue Apr 10 17:15:29 EEST 2007


On Mon, 2007-04-09 at 09:07 -0400, Justin McAleer wrote:

> Moving the dotlock creation after a successful call to 
> mail_storage_create_with_data makes everything happy, but I defer to
> you 
> whether it's safe to call that before locking. 

Hmm. I guess that'd work. I'm not touching v1.0 anymore, but done to CVS
HEAD: http://dovecot.org/list/dovecot-cvs/2007-April/008624.html

> Here's the diff:

Please use diff -u whenever sending diffs. The default diff format is
horrible to read.

Anyway I think it's not really correct, because this needs to be done
after locking (although it's pretty unlikely to happen):

	/* just in case if another process just had converted the mailbox,
	   reopen the source storage */
	mail_storage_destroy(&source_ns->storage);
	if (mail_storage_create(source_ns, NULL, source_data, user,
				src_flags, lock_method) < 0) {
		/* No need for conversion anymore. */
		file_dotlock_delete(&dotlock);
		return 0;
	}

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://dovecot.org/pipermail/dovecot/attachments/20070410/bb5958ac/attachment.pgp 


More information about the dovecot mailing list