diff options
author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-01-08 10:57:34 +0000 |
---|---|---|
committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-01-08 10:58:50 +0000 |
commit | 5f5a51a6c81a50c67bf7c512d548018ff4bf2e58 (patch) | |
tree | 2386816beaa66a1bee54c79cfd425bdfab09dcc6 | |
parent | Merge pull request #3019 (diff) | |
download | monero-5f5a51a6c81a50c67bf7c512d548018ff4bf2e58.tar.xz |
util: warn if unbound was not built with threads
This can causes crashes in libunbound
Diffstat (limited to '')
-rw-r--r-- | src/common/util.cpp | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/src/common/util.cpp b/src/common/util.cpp index 2a2f50c4f..74f33fbb6 100644 --- a/src/common/util.cpp +++ b/src/common/util.cpp @@ -34,6 +34,8 @@ #include <gnu/libc-version.h> #endif +#include "unbound.h" + #include "include_base_utils.h" #include "file_io_utils.h" #include "wipeable_string.h" @@ -521,6 +523,18 @@ std::string get_nix_version_display_string() return std::error_code(code, std::system_category()); } + static bool unbound_built_with_threads() + { + ub_ctx *ctx = ub_ctx_create(); + if (!ctx) return false; // cheat a bit, should not happen unless OOM + ub_ctx_zone_add(ctx, "monero", "unbound"); // this calls ub_ctx_finalize first, then errors out with UB_SYNTAX + // if no threads, bails out early with UB_NOERROR, otherwise fails with UB_AFTERFINAL id already finalized + bool with_threads = ub_ctx_async(ctx, 1) != 0; // UB_AFTERFINAL is not defined in public headers, check any error + ub_ctx_delete(ctx); + MINFO("libunbound was built " << (with_threads ? "with" : "without") << " threads"); + return with_threads; + } + bool sanitize_locale() { // boost::filesystem throws for "invalid" locales, such as en_US.UTF-8, or kjsdkfs, @@ -563,6 +577,9 @@ std::string get_nix_version_display_string() OPENSSL_init_ssl(0, NULL); #endif + if (!unbound_built_with_threads()) + MCLOG_RED(el::Level::Warning, "global", "libunbound was not built with threads enabled - crashes may occur"); + return true; } void set_strict_default_file_permissions(bool strict) |