X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/ede3a6d68af66772b4f5f94208b4126bab566cc8..f52f37959dd8daefc573fc977bbb9aa0810410e5:/src/unix/stackwalk.cpp?ds=inline diff --git a/src/unix/stackwalk.cpp b/src/unix/stackwalk.cpp index 9ad52db9e6..6c234adbea 100644 --- a/src/unix/stackwalk.cpp +++ b/src/unix/stackwalk.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: msw/stackwalk.cpp +// Name: src/unix/stackwalk.cpp // Purpose: wxStackWalker implementation for Unix/glibc // Author: Vadim Zeitlin // Modified by: @@ -33,6 +33,7 @@ #endif #include "wx/stackwalk.h" +#include "wx/stdpaths.h" #include @@ -66,41 +67,38 @@ public: private: FILE *m_fp; - DECLARE_NO_COPY_CLASS(wxStdioPipe) + wxDECLARE_NO_COPY_CLASS(wxStdioPipe); }; // ============================================================================ // implementation // ============================================================================ -wxString wxStackWalker::ms_exepath; - // ---------------------------------------------------------------------------- // wxStackFrame // ---------------------------------------------------------------------------- void wxStackFrame::OnGetName() { - if ( m_hasName ) + if ( !m_name.empty() ) return; - m_hasName = true; - - // try addr2line first because it always gives us demangled names (even if - // __cxa_demangle is not available) and because it seems less error-prone + // we already tried addr2line in wxStackWalker::InitFrames: it always + // gives us demangled names (even if __cxa_demangle is not available) when + // the function is part of the ELF (when it's in a shared object addr2line + // will give "??") and because it seems less error-prone. // when it works, backtrace_symbols() sometimes returns incorrect results - OnGetLocation(); // format is: "module(funcname+offset) [address]" but the part in // parentheses can be not present wxString syminfo = wxString::FromAscii(m_syminfo); - const size_t posOpen = syminfo.find(_T('(')); + const size_t posOpen = syminfo.find(wxT('(')); if ( posOpen != wxString::npos ) { - const size_t posPlus = syminfo.find(_T('+'), posOpen + 1); + const size_t posPlus = syminfo.find(wxT('+'), posOpen + 1); if ( posPlus != wxString::npos ) { - const size_t posClose = syminfo.find(_T(')'), posPlus + 1); + const size_t posClose = syminfo.find(wxT(')'), posPlus + 1); if ( posClose != wxString::npos ) { if ( m_name.empty() ) @@ -129,103 +127,183 @@ void wxStackFrame::OnGetName() m_offset = ofs; } } + + m_module.assign(syminfo, posOpen); } + else // not in "module(funcname+offset)" format + { + m_module = syminfo; + } +} + + +// ---------------------------------------------------------------------------- +// wxStackWalker +// ---------------------------------------------------------------------------- + +// that many frames should be enough for everyone +#define MAX_FRAMES 200 + +// we need a char buffer big enough to contain a call to addr2line with +// up to MAX_FRAMES addresses ! +// NB: %p specifier will print the pointer in hexadecimal form +// and thus will require 2 chars for each byte + 3 for the +// " 0x" prefix +#define CHARS_PER_FRAME (sizeof(void*) * 2 + 3) + +// BUFSIZE will be 2250 for 32 bit machines +#define BUFSIZE (50 + MAX_FRAMES*CHARS_PER_FRAME) - m_module.assign(syminfo, posOpen); +// static data +void *wxStackWalker::ms_addresses[MAX_FRAMES]; +char **wxStackWalker::ms_symbols = NULL; +int wxStackWalker::m_depth = 0; +wxString wxStackWalker::ms_exepath; +static char g_buf[BUFSIZE]; + + +void wxStackWalker::SaveStack(size_t maxDepth) +{ + // read all frames required + maxDepth = wxMin(WXSIZEOF(ms_addresses)/sizeof(void*), maxDepth); + m_depth = backtrace(ms_addresses, maxDepth*sizeof(void*)); + if ( !m_depth ) + return; + + ms_symbols = backtrace_symbols(ms_addresses, m_depth); } -void wxStackFrame::OnGetLocation() +void wxStackWalker::ProcessFrames(size_t skip) { - if ( m_hasLocation ) + wxStackFrame frames[MAX_FRAMES]; + + if (!ms_symbols || !m_depth) return; - m_hasLocation = true; + // we have 3 more "intermediate" frames which the calling code doesn't know + // about, account for them + skip += 3; + + // call addr2line only once since this call may be very slow + // (it has to load in memory the entire EXE of this app which may be quite + // big, especially if it contains debug info and is compiled statically!) + int towalk = InitFrames(frames, m_depth - skip, &ms_addresses[skip], &ms_symbols[skip]); + // now do user-defined operations on each frame + for ( int n = 0; n < towalk - (int)skip; n++ ) + OnStackFrame(frames[n]); +} + +void wxStackWalker::FreeStack() +{ + // ms_symbols has been allocated by backtrace_symbols() and it's the responsibility + // of the caller, i.e. us, to free that pointer + if (ms_symbols) + free( ms_symbols ); + ms_symbols = NULL; + m_depth = 0; +} + +int wxStackWalker::InitFrames(wxStackFrame *arr, size_t n, void **addresses, char **syminfo) +{ // we need to launch addr2line tool to get this information and we need to // have the program name for this wxString exepath = wxStackWalker::GetExePath(); if ( exepath.empty() ) { - if ( !wxTheApp || !wxTheApp->argv ) - return; - exepath = wxTheApp->argv[0]; + exepath = wxStandardPaths::Get().GetExecutablePath(); + if ( exepath.empty() ) + { + wxLogDebug(wxT("Cannot parse stack frame because the executable ") + wxT("path could not be detected")); + return 0; + } } - wxStdioPipe fp(wxString::Format(_T("addr2line -C -f -e \"%s\" %p"), - exepath.c_str(), m_address).mb_str(), - "r"); - - if ( !fp ) - return; - - // parse addr2line output - char buf[1024]; - if ( !fgets(buf, WXSIZEOF(buf), fp) ) + // build the (long) command line for executing addr2line in an optimized way + // (e.g. use always chars, even in Unicode build: popen() always takes chars) + int len = snprintf(g_buf, BUFSIZE, "addr2line -C -f -e \"%s\"", (const char*) exepath.mb_str()); + len = (len <= 0) ? strlen(g_buf) : len; // in case snprintf() is broken + for (size_t i=0; i