mboxlist_createmailbox_full() in http_carddav.c

Ken Murchison murch at andrew.cmu.edu
Sat Jul 19 12:07:02 EDT 2014

Hi Johan,

Good catch.  This a result of me always testing in a Murder environment 
for use at CMU.

I just committed this fix: 

On 07/18/2014 02:34 PM, Johan Hattne wrote:
> Dear all;
> I’m trying to understand what’s happening in the my_carddav_auth() function in http_carddav.c, in particular the part that creates carddav mailboxes.  If I understand correctly, on the user’s first successful authentication, mboxlist_lookup() will return r = IMAP_MAILBOX_NONEXISTENT, and program goes into the if-clause (line ~362 in the beta9-code I’m looking at).  Unless there’s an mupdate-server, the subsequent call to mboxlist_createmailbox_full() will not happen because !r is false, and no mailboxes would be created.  I suppose this isn’t what was intended—clues?
> // Johan
> ----
> Cyrus Home Page: http://www.cyrusimap.org/
> List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
> To Unsubscribe:
> https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus

Kenneth Murchison
Principal Systems Software Engineer
Carnegie Mellon University

More information about the Info-cyrus mailing list