From 48595674fa78ebeb55035aca4c439bb39385031a Mon Sep 17 00:00:00 2001 From: Antoine Poinsot Date: Tue, 19 Jan 2021 11:30:45 +0100 Subject: [PATCH] hsmtool: don't streq() on NULL This would cause a segfault on the default network parameter for `dumponchaindescriptors`. Introduced in 1513a2d07e Changelog-Fixed: hsmtool: fix a segfault on `dumponchaindescriptors` without network parameter Signed-off-by: Antoine Poinsot --- tools/hsmtool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/hsmtool.c b/tools/hsmtool.c index af8a73deb..3bc076366 100644 --- a/tools/hsmtool.c +++ b/tools/hsmtool.c @@ -660,9 +660,9 @@ int main(int argc, char *argv[]) if (deprecated_apis && argc > 4) net = argv[4]; - if (streq(net, "testnet")) + if (net && streq(net, "testnet")) is_testnet = true; - else if (!streq(net, "bitcoin")) + else if (net && !streq(net, "bitcoin")) errx(ERROR_USAGE, "Network '%s' not supported." " Supported networks: bitcoin (default)," " testnet", net);