> Version 2...
> Part 3 of 3 of this patch with the request changes to options.c
> is now available for review. The full details are located here:
> The revised patch has been updated to include all changes as on
> including the commit of:
> Modified Files:
> cyrus.c 1.159 -> 1.160
> The only file modified between the first and second version of this patch
> was options.c
Looking at the patch to abandon.c, first chunk
it looks like you'll deadlock on the ld_req_mutex, based on the preceding
comment. I.e., you should not be using LDAP_NEXT_MSGID() here.
Never mind, I forgot that LDAP_NEXT_MSGID uses its own msgid mutex now.
-- Howard Chu
CTO, Symas Corp.