davide/palladum-lightning
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity

common: don't crash on send_backtrace() if bt_print is NULL.

Browse Source 
This (and bt_exit) are NULL for libplugin.  We don't always want to crash!

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is contained in:
Rusty Russell
2025-01-23 09:38:29 +10:30
parent fd7ac4e05b
commit 89ca5aef9f
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
common/daemon.c
View File

@@ -32,6 +32,9 @@ void send_backtrace(const char *why)
if (backtrace_state) if (backtrace_state)
backtrace_print(backtrace_state, 0, stderr); backtrace_print(backtrace_state, 0, stderr);
if (!bt_print)
return;
/* Now send to parent. */ /* Now send to parent. */
bt_print("%s (version %s)", why, version()); bt_print("%s (version %s)", why, version());
if (backtrace_state) if (backtrace_state)
@@ -75,7 +78,8 @@ static void crashdump(int sig)
send_backtrace(why); send_backtrace(why);
/* Probably shouldn't return. */ /* Probably shouldn't return. */
bt_exit(); if (bt_exit)
bt_exit();
/* This time it will kill us instantly. */ /* This time it will kill us instantly. */
kill(getpid(), sig); kill(getpid(), sig);