> On Wed, May 24, 2017, at 04:17 PM, Don Lewis wrote: >> This first version of the patch worked for me. A problem with >> >> the second version of the patch is that mlookup() gets called in >> >> a bunch of different places and the server variable would need >> >> to be initialized in all of them. In verify_user(), mlookup is called >> >> several times, so server would have to be set to NULL before >> >> each call. >> >> > > Oh, good catch, I didn't look for other occurrences. > > Attached is an updated version of my patch that initialises these > variables correctly. Thanks, I can confirm that the updated patch fixes the issue for me. Kind regards, Simon ---- 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