[PATCH 06/13] Fix sync of non-user mailboxes

Wesley Craig wes at umich.edu
Wed Jan 28 19:38:01 EST 2009


On 27 Jan 2009, at 23:15, Bron Gondwana wrote:
> If a non-user mailbox is being considered for promotion to a USER
> event in sync_client, it trys to xstrdup the folder name starting
> at the user name - except that's a NULL string and it segfaults.
>
> This patch tests the result of mboxname_isusermailbox in the
> conditional instead, and then makes the copy only if it's non-NULL.

Looks good.  It would be cool if it had a BZ number, since it's  
pretty obviously a bug.

:wes


More information about the Cyrus-devel mailing list