From: Armin Novak Date: Mon, 17 Dec 2018 08:09:02 +0000 (+0100) Subject: Added backtrace function working on allocated logger. X-Git-Tag: 2.0.0~567^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5dea64a468b6ffa22d4a1606c174d16e72a4e8b7;p=platform%2Fupstream%2Ffreerdp.git Added backtrace function working on allocated logger. --- diff --git a/winpr/include/winpr/debug.h b/winpr/include/winpr/debug.h index 8faa999..45ed778 100644 --- a/winpr/include/winpr/debug.h +++ b/winpr/include/winpr/debug.h @@ -27,8 +27,10 @@ extern "C" { #include #include +#include WINPR_API void winpr_log_backtrace(const char* tag, DWORD level, DWORD size); +WINPR_API void winpr_log_backtrace_ex(wLog* log, DWORD level, DWORD size); WINPR_API void* winpr_backtrace(DWORD size); WINPR_API void winpr_backtrace_free(void* buffer); WINPR_API char** winpr_backtrace_symbols(void* buffer, size_t* used); diff --git a/winpr/libwinpr/utils/debug.c b/winpr/libwinpr/utils/debug.c index 8196d04..ae19923 100644 --- a/winpr/libwinpr/utils/debug.c +++ b/winpr/libwinpr/utils/debug.c @@ -53,12 +53,12 @@ #define LOGE(...) do { WLog_Print(WLog_Get(TAG), WLOG_ERROR, __VA_ARGS__); } while(0) #define LOGF(...) do { WLog_Print(WLog_Get(TAG), WLOG_FATAL, __VA_ARGS__); } while(0) -static const char *support_msg = "Invalid stacktrace buffer! check if platform is supported!"; +static const char* support_msg = "Invalid stacktrace buffer! check if platform is supported!"; #if defined(HAVE_EXECINFO_H) typedef struct { - void **buffer; + void** buffer; size_t max; size_t used; } t_execinfo; @@ -67,9 +67,9 @@ typedef struct #if defined(_WIN32) || defined(_WIN64) typedef struct { - PVOID* stack; - ULONG used; - ULONG max; + PVOID* stack; + ULONG used; + ULONG max; } t_win_stack; #endif @@ -80,31 +80,31 @@ typedef struct typedef struct { - backtrace_frame_t *buffer; + backtrace_frame_t* buffer; size_t max; size_t used; } t_corkscrew_data; typedef struct { - void *hdl; - ssize_t (*unwind_backtrace)(backtrace_frame_t *backtrace, size_t ignore_depth, size_t max_depth); - ssize_t (*unwind_backtrace_thread)(pid_t tid, backtrace_frame_t *backtrace, - size_t ignore_depth, size_t max_depth); - ssize_t (*unwind_backtrace_ptrace)(pid_t tid, const ptrace_context_t *context, - backtrace_frame_t *backtrace, size_t ignore_depth, size_t max_depth); - void (*get_backtrace_symbols)(const backtrace_frame_t *backtrace, size_t frames, - backtrace_symbol_t *backtrace_symbols); - void (*get_backtrace_symbols_ptrace)(const ptrace_context_t *context, - const backtrace_frame_t *backtrace, size_t frames, - backtrace_symbol_t *backtrace_symbols); - void (*free_backtrace_symbols)(backtrace_symbol_t *backtrace_symbols, size_t frames); - void (*format_backtrace_line)(unsigned frameNumber, const backtrace_frame_t *frame, - const backtrace_symbol_t *symbol, char *buffer, size_t bufferSize); + void* hdl; + ssize_t (*unwind_backtrace)(backtrace_frame_t* backtrace, size_t ignore_depth, size_t max_depth); + ssize_t (*unwind_backtrace_thread)(pid_t tid, backtrace_frame_t* backtrace, + size_t ignore_depth, size_t max_depth); + ssize_t (*unwind_backtrace_ptrace)(pid_t tid, const ptrace_context_t* context, + backtrace_frame_t* backtrace, size_t ignore_depth, size_t max_depth); + void (*get_backtrace_symbols)(const backtrace_frame_t* backtrace, size_t frames, + backtrace_symbol_t* backtrace_symbols); + void (*get_backtrace_symbols_ptrace)(const ptrace_context_t* context, + const backtrace_frame_t* backtrace, size_t frames, + backtrace_symbol_t* backtrace_symbols); + void (*free_backtrace_symbols)(backtrace_symbol_t* backtrace_symbols, size_t frames); + void (*format_backtrace_line)(unsigned frameNumber, const backtrace_frame_t* frame, + const backtrace_symbol_t* symbol, char* buffer, size_t bufferSize); } t_corkscrew; static pthread_once_t initialized = PTHREAD_ONCE_INIT; -static t_corkscrew *fkt = NULL; +static t_corkscrew* fkt = NULL; void load_library(void) { @@ -189,14 +189,16 @@ fail: #if defined(_WIN32) && (NTDDI_VERSION <= NTDDI_WINXP) -typedef USHORT (WINAPI * PRTL_CAPTURE_STACK_BACK_TRACE_FN)(ULONG FramesToSkip, ULONG FramesToCapture, PVOID* BackTrace, PULONG BackTraceHash); +typedef USHORT(WINAPI* PRTL_CAPTURE_STACK_BACK_TRACE_FN)(ULONG FramesToSkip, ULONG FramesToCapture, + PVOID* BackTrace, PULONG BackTraceHash); static HMODULE g_NTDLL_Library = NULL; static BOOL g_RtlCaptureStackBackTrace_Detected = FALSE; static BOOL g_RtlCaptureStackBackTrace_Available = FALSE; static PRTL_CAPTURE_STACK_BACK_TRACE_FN g_pRtlCaptureStackBackTrace = NULL; -USHORT RtlCaptureStackBackTrace(ULONG FramesToSkip, ULONG FramesToCapture, PVOID* BackTrace, PULONG BackTraceHash) +USHORT RtlCaptureStackBackTrace(ULONG FramesToSkip, ULONG FramesToCapture, PVOID* BackTrace, + PULONG BackTraceHash) { if (!g_RtlCaptureStackBackTrace_Detected) { @@ -204,7 +206,8 @@ USHORT RtlCaptureStackBackTrace(ULONG FramesToSkip, ULONG FramesToCapture, PVOID if (g_NTDLL_Library) { - g_pRtlCaptureStackBackTrace = (PRTL_CAPTURE_STACK_BACK_TRACE_FN) GetProcAddress(g_NTDLL_Library, "RtlCaptureStackBackTrace"); + g_pRtlCaptureStackBackTrace = (PRTL_CAPTURE_STACK_BACK_TRACE_FN) GetProcAddress(g_NTDLL_Library, + "RtlCaptureStackBackTrace"); g_RtlCaptureStackBackTrace_Available = (g_pRtlCaptureStackBackTrace) ? TRUE : FALSE; } else @@ -235,19 +238,15 @@ void winpr_backtrace_free(void* buffer) #if defined(HAVE_EXECINFO_H) t_execinfo* data = (t_execinfo*) buffer; - free(data->buffer); - free(data); #elif defined(ANDROID) - t_corkscrew_data *data = (t_corkscrew_data *)buffer; - + t_corkscrew_data* data = (t_corkscrew_data*)buffer; free(data->buffer); - free(data); #elif defined(_WIN32) || defined(_WIN64) { - t_win_stack *data = (t_win_stack*)buffer; + t_win_stack* data = (t_win_stack*)buffer; free(data->stack); free(data); } @@ -311,7 +310,6 @@ void* winpr_backtrace(DWORD size) SymInitialize(process, NULL, TRUE); data->used = RtlCaptureStackBackTrace(2, size, data->stack, NULL); - return data; #else LOGF(support_msg); @@ -359,8 +357,7 @@ char** winpr_backtrace_symbols(void* buffer, size_t* used) size_t i; size_t array_size = data->used * sizeof(char*); size_t lines_size = data->used * line_len; - char **vlines = calloc(1, array_size + lines_size); - + char** vlines = calloc(1, array_size + lines_size); backtrace_symbol_t* symbols = calloc(data->used, sizeof(backtrace_symbol_t)); if (!vlines || !symbols) @@ -376,7 +373,7 @@ char** winpr_backtrace_symbols(void* buffer, size_t* used) fkt->get_backtrace_symbols(data->buffer, data->used, symbols); - for (i = 0; i used; i++) + for (i = 0; i < data->used; i++) fkt->format_backtrace_line(i, &data->buffer[i], &symbols[i], vlines[i], line_len); fkt->free_backtrace_symbols(symbols, data->used); @@ -387,6 +384,7 @@ char** winpr_backtrace_symbols(void* buffer, size_t* used) return vlines; } + #elif (defined(_WIN32) || defined(_WIN64)) && !defined(_UWP) { size_t i; @@ -395,16 +393,16 @@ char** winpr_backtrace_symbols(void* buffer, size_t* used) t_win_stack* data = (t_win_stack*) buffer; size_t array_size = data->used * sizeof(char*); size_t lines_size = data->used * line_len; - char **vlines = calloc(1, array_size + lines_size); + char** vlines = calloc(1, array_size + lines_size); SYMBOL_INFO* symbol = calloc(1, sizeof(SYMBOL_INFO) + line_len * sizeof(char)); IMAGEHLP_LINE64* line = (IMAGEHLP_LINE64*) calloc(1, sizeof(IMAGEHLP_LINE64)); if (!vlines || !symbol || !line) { - free(vlines); - free(symbol); - free(line); - return NULL; + free(vlines); + free(symbol); + free(line); + return NULL; } line->SizeOfStruct = sizeof(IMAGEHLP_LINE64); @@ -419,24 +417,23 @@ char** winpr_backtrace_symbols(void* buffer, size_t* used) { DWORD64 address = (DWORD64)(data->stack[i]); DWORD displacement; - SymFromAddr(process, address, 0, symbol); if (SymGetLineFromAddr64(process, address, &displacement, line)) { - sprintf_s(vlines[i], line_len, "%016"PRIx64": %s in %s:%"PRIu32, symbol->Address, symbol->Name, line->FileName, line->LineNumber); + sprintf_s(vlines[i], line_len, "%016"PRIx64": %s in %s:%"PRIu32, symbol->Address, symbol->Name, + line->FileName, line->LineNumber); } else sprintf_s(vlines[i], line_len, "%016"PRIx64": %s", symbol->Address, symbol->Name); - } - - if (used) - *used = data->used; + } - free(symbol); - free(line); + if (used) + *used = data->used; - return vlines; + free(symbol); + free(line); + return vlines; } #else LOGF(support_msg); @@ -464,7 +461,6 @@ void winpr_backtrace_symbols_fd(void* buffer, int fd) DWORD i; size_t used; char** lines; - lines = winpr_backtrace_symbols(buffer, &used); if (lines) @@ -480,23 +476,30 @@ void winpr_backtrace_symbols_fd(void* buffer, int fd) void winpr_log_backtrace(const char* tag, DWORD level, DWORD size) { + winpr_log_backtrace_ex(WLog_Get(tag), level, size); +} + +void winpr_log_backtrace_ex(wLog* log, DWORD level, DWORD size) +{ size_t used, x; - char **msg; - void *stack = winpr_backtrace(20); + char** msg; + void* stack = winpr_backtrace(20); if (!stack) { - WLog_ERR(tag, "winpr_backtrace failed!\n"); + WLog_Print(log, WLOG_ERROR, "winpr_backtrace failed!\n"); winpr_backtrace_free(stack); return; } msg = winpr_backtrace_symbols(stack, &used); + if (msg) { - for (x=0; x