diff options
| author | Adam Langley <agl@google.com> | 2015-01-31 04:08:37 +0300 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2015-02-02 22:14:15 +0300 |
| commit | 2b2d66d4094e5c25789ce68310d86192c377c63b (patch) | |
| tree | 0037574fca9fad9755016ccc0f8e4ddb0f978a59 /crypto/thread.c | |
| parent | efed2210e8e0c6b3db6c63e354f0c9b67673c712 (diff) | |
Remove string.h from base.h.
Including string.h in base.h causes any file that includes a BoringSSL
header to include string.h. Generally this wouldn't be a problem,
although string.h might slow down the compile if it wasn't otherwise
needed. However, it also causes problems for ipsec-tools in Android
because OpenSSL didn't have this behaviour.
This change removes string.h from base.h and, instead, adds it to each
.c file that requires it.
Change-Id: I5968e50b0e230fd3adf9b72dd2836e6f52d6fb37
Reviewed-on: https://boringssl-review.googlesource.com/3200
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'crypto/thread.c')
| -rw-r--r-- | crypto/thread.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/crypto/thread.c b/crypto/thread.c index 6e8d4949..f18bc139 100644 --- a/crypto/thread.c +++ b/crypto/thread.c @@ -57,6 +57,7 @@ #include <openssl/thread.h> #include <errno.h> +#include <string.h> #if defined(OPENSSL_WINDOWS) #pragma warning(push, 3) @@ -67,6 +68,7 @@ #include <openssl/mem.h> #include <openssl/type_check.h> + /* lock_names contains the names of all the locks defined in thread.h. */ static const char *const lock_names[] = { "<<ERROR>>", "err", "ex_data", "x509", |