summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStjepan Rajko <stjepanr@axosoft.com>2016-02-25 15:19:51 -0700
committerStjepan Rajko <stjepanr@axosoft.com>2016-02-26 15:31:24 -0700
commit09db7fd8a6b9b20588b1a42342eda4f0edbbffea (patch)
tree62a96728197c5bfbae40ef51fe5e5d44eefbb94e
parent9f4e7c84904030ecc7d14637fa8c51bac087fedf (diff)
downloadlibgit2-09db7fd8a6b9b20588b1a42342eda4f0edbbffea.tar.gz
Expand OpenSSL and libssh2 thread safety documentation
-rw-r--r--THREADING.md16
1 files changed, 11 insertions, 5 deletions
diff --git a/THREADING.md b/THREADING.md
index 3717d6c88..0b9e50286 100644
--- a/THREADING.md
+++ b/THREADING.md
@@ -72,13 +72,19 @@ which locking function it should use. This means that libgit2 cannot
know what to set as the user of libgit2 may use OpenSSL independently
and the locking settings must survive libgit2 shutting down.
+Even if libgit2 doesn't use OpenSSL directly, OpenSSL can still be used
+by libssh2 depending on the configuration. If OpenSSL is used both by
+libgit2 and libssh2, you only need to set up threading for OpenSSL once.
+
libgit2 does provide a last-resort convenience function
`git_openssl_set_locking()` (available in `sys/openssl.h`) to use the
platform-native mutex mechanisms to perform the locking, which you may
rely on if you do not want to use OpenSSL outside of libgit2, or you
know that libgit2 will outlive the rest of the operations. It is not
safe to use OpenSSL multi-threaded after libgit2's shutdown function
-has been called.
+has been called. Note `git_openssl_set_locking()` only works if
+libgit2 uses OpenSSL directly - if OpenSSL is only used as a dependency
+of libssh2 as described above, `git_openssl_set_locking()` is a no-op.
If your programming language offers a package/bindings for OpenSSL,
you should very strongly prefer to use that in order to set up
@@ -87,14 +93,14 @@ when using this function.
See the
[OpenSSL documentation](https://www.openssl.org/docs/crypto/threads.html)
-on threading for more details.
+on threading for more details, and http://trac.libssh2.org/wiki/MultiThreading
+for a specific example of providing the threading callbacks.
Be also aware that libgit2 does not always link against OpenSSL
if there are alternatives provided by the system.
-libssh2 may be linked against OpenSSL or libgcrypt. If it uses
-OpenSSL, you only need to set up threading for OpenSSL once and the
-above paragraphs are enough. If it uses libgcrypt, then you need to
+libssh2 may be linked against OpenSSL or libgcrypt. If it uses OpenSSL,
+see the above paragraphs. If it uses libgcrypt, then you need to
set up its locking before using it multi-threaded. libgit2 has no
direct connection to libgcrypt and thus has not convenience functions for
it (but libgcrypt has macros). Read libgcrypt's