Bug #4949

deleteMailbox() tries to reset ACL on deleted mailbox

Details

Submitted	2005-07-29 22:38 UTC
From	patrick at coverallcrew dot com
Assigned	amistry
Status	Closed
Package	Net_Cyrus
PHP Version	4.3.11
OS	FreeBSD
Roadmaps	(Not assigned)

Comments

[2005-07-29 22:38 UTC] patrick at coverallcrew dot com

Description:
------------
On line 345 of Net_Cyrus.php, there is a call to
_resetAdminPriv() on the mailbox that was just deleted. This
line should be moved to line 343, right before a PEAR_Error is
returned, as the only time you would want to reset the
privileges is if the mailbox could not be deleted.

[2006-01-27 22:36 UTC] amistry at php dot net

I know this is trivial, but could you just verify this patch works for you.
http://am-productions.biz/docs/Cyrus.patch

[2006-01-27 22:52 UTC] patrick at coverallcrew dot com

Much better. (This patch changes the code to what I already
had changed myself, so I know it fixes it.)

[2006-02-03 21:18 UTC] amistry at php dot net

Thanks, I'm going to ask for the maintainer hat for this package and will try to update it as soon as I can. It may take about a month or so since I'll be out of town for a few weeks. If you don't hear anything by mid-March send me a reminder.

[2006-03-22 04:06 UTC] amistry at php dot net

Fixed in the CVS.