* Heikki Linnakangas (hlinnakan...@vmware.com) wrote: > Actually, I think there's a pre-existing bug there in git master. If > the SSL_set_app_data or SSL_set_fd call in pqsecure_open_client > fails for some reason, it will call close_SSL() with conn->ssl > already set, and the mutex held. close_SSL() will call > pqsecure_destroy()->destroy_SSL()->destory_ssl_system(), which will > try to acquire the mutex again.
Yeah, good point, and that one looks like my fault. Moving it after the unlock should address that tho, no? Looking through the other close_SSL() calls, I don't see any other situations where it might be called with the lock held. Thanks, Stephen
signature.asc
Description: Digital signature