cyrus-imapd bug(?) and possible bug-fix

[Ken Murchison]

Aristotelis wrote:
>   
>    Hello all,
> 
>  A bug[1] in cyrus imapd was reported to me[2] by Patrick T. Tsang, that has
> to do with renaming folder when using unix hierarchy separators, altnamespace and
> virtual domains. The problem is illustrated in the following session:
> 
> . login arisg at edunet.gr HiI'mAPassowd 
> . OK User logged in
> . list * *
> * LIST (\Noinferiors) "/" "INBOX"
> * LIST (\HasNoChildren) "/" "Drafts"
> * LIST (\HasChildren) "/" "FolderA"
> * LIST (\HasChildren) "/" "FolderA/FolderB"
> * LIST (\HasNoChildren) "/" "FolderA/FolderB/FolderC"
> * LIST (\HasNoChildren) "/" "Sent"
> * LIST (\HasNoChildren) "/" "Templates"
> * LIST (\HasNoChildren) "/" "Trash"
> * LIST (\HasNoChildren) "/" "lala"
> . OK Completed (0.000 secs 10 calls)
> . rename FolderA/FolderB FolderA/FolderBB
> * OK rename Other Users/arisg/FolderA/FolderB Other Users/arisg/FolderA/FolderBB
> . OK Completed
> 
> 
>  Actually there should be two things different IMHO. There shouldn't
> be the Other Users part, and FolderA/FolderB/FolderC should also
> be renamed in FolderA/FolderBB/FolderC.

Thanks for the report and diagnostics.  Just committed the following 
patch to CVS:


Modified Files:
	mboxlist.c mboxname.c
Log Message:
fixed bug with RENAME and virtdomains+altnamespace (based on patch from
Aristotelis <arisg at noc.uoa.gr>)


--- links to diffs follow ---
http://bugzilla.andrew.cmu.edu/cvsweb/src/cyrus/imap/mboxlist.c.diff?r1=1.238&r2=1.239
http://bugzilla.andrew.cmu.edu/cvsweb/src/cyrus/imap/mboxname.c.diff?r1=1.34&r2=1.35


-- 
Kenneth Murchison     Oceana Matrix Ltd.
Software Engineer     21 Princeton Place
716-662-8973 x26      Orchard Park, NY 14127
--PGP Public Key--    http://www.oceana.com/~ken/ksm.pgp
---
Cyrus Home Page: http://asg.web.cmu.edu/cyrus
Cyrus Wiki/FAQ: http://cyruswiki.andrew.cmu.edu
List Archives/Info: http://asg.web.cmu.edu/cyrus/mailing-list.html