Browse Source

Add guard for BACKTRACE_SUPPORTED

ppa-0.6.1
conanoc 7 years ago
committed by Rusty Russell
parent
commit
c4700a13a7
  1. 5
      common/subdaemon.c
  2. 3
      external/Makefile
  3. 7
      lightningd/log.c

5
common/subdaemon.c

@ -1,4 +1,5 @@
#include <backtrace.h> #include <backtrace.h>
#include <backtrace-supported.h>
#include <ccan/err/err.h> #include <ccan/err/err.h>
#include <ccan/str/str.h> #include <ccan/str/str.h>
#include <common/dev_disconnect.h> #include <common/dev_disconnect.h>
@ -12,6 +13,7 @@
#include <sys/types.h> #include <sys/types.h>
#include <unistd.h> #include <unistd.h>
#if BACKTRACE_SUPPORTED
static struct backtrace_state *backtrace_state; static struct backtrace_state *backtrace_state;
static int backtrace_status(void *unused UNUSED, uintptr_t pc, static int backtrace_status(void *unused UNUSED, uintptr_t pc,
@ -51,6 +53,7 @@ static void crashlog_activate(void)
sigaction(SIGSEGV, &sa, NULL); sigaction(SIGSEGV, &sa, NULL);
sigaction(SIGBUS, &sa, NULL); sigaction(SIGBUS, &sa, NULL);
} }
#endif
#if DEVELOPER #if DEVELOPER
extern volatile bool debugger_connected; extern volatile bool debugger_connected;
@ -65,8 +68,10 @@ void subdaemon_setup(int argc, char *argv[])
} }
err_set_progname(argv[0]); err_set_progname(argv[0]);
#if BACKTRACE_SUPPORTED
backtrace_state = backtrace_create_state(argv[0], 0, NULL, NULL); backtrace_state = backtrace_create_state(argv[0], 0, NULL, NULL);
crashlog_activate(); crashlog_activate();
#endif
/* We handle write returning errors! */ /* We handle write returning errors! */
signal(SIGPIPE, SIG_IGN); signal(SIGPIPE, SIG_IGN);

3
external/Makefile

@ -16,7 +16,8 @@ EXTERNAL_INCLUDE_FLAGS := \
-I external/libwally-core/src/secp256k1/include/ \ -I external/libwally-core/src/secp256k1/include/ \
-I external/jsmn/ \ -I external/jsmn/ \
-I external/libbase58/ \ -I external/libbase58/ \
-I external/libbacktrace -I external/libbacktrace/ \
-I external/libbacktrace-build
EXTERNAL_LDLIBS := -Lexternal $(patsubst lib%.a,-l%,$(notdir $(EXTERNAL_LIBS))) EXTERNAL_LDLIBS := -Lexternal $(patsubst lib%.a,-l%,$(notdir $(EXTERNAL_LIBS)))

7
lightningd/log.c

@ -1,5 +1,6 @@
#include "log.h" #include "log.h"
#include <backtrace.h> #include <backtrace.h>
#include <backtrace-supported.h>
#include <ccan/array_size/array_size.h> #include <ccan/array_size/array_size.h>
#include <ccan/list/list.h> #include <ccan/list/list.h>
#include <ccan/opt/opt.h> #include <ccan/opt/opt.h>
@ -463,6 +464,7 @@ void opt_register_logging(struct lightningd *ld)
"log to file instead of stdout"); "log to file instead of stdout");
} }
#if BACKTRACE_SUPPORTED
static int log_backtrace(void *log, uintptr_t pc, static int log_backtrace(void *log, uintptr_t pc,
const char *filename, int lineno, const char *filename, int lineno,
const char *function) const char *function)
@ -521,9 +523,11 @@ static void log_crash(int sig)
fprintf(stderr, "Log dumped in %s", logfile); fprintf(stderr, "Log dumped in %s", logfile);
fprintf(stderr, "\n"); fprintf(stderr, "\n");
} }
#endif
void crashlog_activate(const char *argv0 UNUSED, struct log *log) void crashlog_activate(const char *argv0 UNUSED, struct log *log)
{ {
#if BACKTRACE_SUPPORTED
struct sigaction sa; struct sigaction sa;
crashlog = log; crashlog = log;
@ -536,6 +540,7 @@ void crashlog_activate(const char *argv0 UNUSED, struct log *log)
sigaction(SIGFPE, &sa, NULL); sigaction(SIGFPE, &sa, NULL);
sigaction(SIGSEGV, &sa, NULL); sigaction(SIGSEGV, &sa, NULL);
sigaction(SIGBUS, &sa, NULL); sigaction(SIGBUS, &sa, NULL);
#endif
} }
void log_dump_to_file(int fd, const struct log_book *lr) void log_dump_to_file(int fd, const struct log_book *lr)
@ -575,6 +580,7 @@ void fatal(const char *fmt, ...)
fprintf(stderr, "\n"); fprintf(stderr, "\n");
va_end(ap); va_end(ap);
#if BACKTRACE_SUPPORTED
/* Early on, we just dump errors to stderr. */ /* Early on, we just dump errors to stderr. */
if (crashlog) { if (crashlog) {
va_start(ap, fmt); va_start(ap, fmt);
@ -582,6 +588,7 @@ void fatal(const char *fmt, ...)
va_end(ap); va_end(ap);
log_crash(0); log_crash(0);
} }
#endif
abort(); abort();
} }

Loading…
Cancel
Save