summaryrefslogtreecommitdiff
path: root/indra/llcommon
diff options
context:
space:
mode:
authorGraham Linden <graham@lindenlab.com>2018-06-26 19:09:09 +0100
committerGraham Linden <graham@lindenlab.com>2018-06-26 19:09:09 +0100
commit3cd2a35f9db411616ad15b89c8af366c0ccf00c9 (patch)
tree55970688962261b5c9d0b5ae1d2e4f675dfb878b /indra/llcommon
parent520ba7fd214b372e61d3e48d12bb1baab1c59d48 (diff)
Crib OSX 10.14 fix for MAINT-8724
Diffstat (limited to 'indra/llcommon')
-rw-r--r--indra/llcommon/llerror.cpp24
1 files changed, 7 insertions, 17 deletions
diff --git a/indra/llcommon/llerror.cpp b/indra/llcommon/llerror.cpp
index f31a054139..29de79dc64 100644
--- a/indra/llcommon/llerror.cpp
+++ b/indra/llcommon/llerror.cpp
@@ -249,23 +249,13 @@ namespace LLError
{
#ifdef __GNUC__
// GCC: type_info::name() returns a mangled class name,st demangle
-
- static size_t abi_name_len = 100;
- static char* abi_name_buf = (char*)malloc(abi_name_len);
- // warning: above is voodoo inferred from the GCC manual,
- // do NOT change
-
- int status;
- // We don't use status, and shouldn't have to pass apointer to it
- // but gcc 3.3 libstc++'s implementation of demangling is broken
- // and fails without.
-
- char* name = abi::__cxa_demangle(mangled,
- abi_name_buf, &abi_name_len, &status);
- // this call can realloc the abi_name_buf pointer (!)
-
- return name ? name : mangled;
-
+ // passing nullptr, 0 forces allocation of a unique buffer we can free
+ // fixing MAINT-8724 on OSX 10.14
+ int status = -1;
+ char* name = abi::__cxa_demangle(mangled, nullptr, 0, &status);
+ std::string result(name ? name : mangled);
+ free(name);
+ return result;
#elif LL_WINDOWS
// DevStudio: type_info::name() includes the text "class " at the start