mboxlist_createmailbox_full() in http_carddav.c

[Ken Murchison]

Hi Johan,

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

I just committed this fix: 
http://git.cyrusimap.org/cyrus-imapd/commit/?h=caldav-2.4&id=72dd0606feac1abf798a94448434a2d9013e490a


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