Skip to content

Commit

Permalink
plugin: Add missing context to tal_fmt call on error message
Browse files Browse the repository at this point in the history
Seems the context parameter got lost somewhere.

Reported-by: Ulmo <@Ulmo>
Signed-off-by: Christian Decker <decker.christian@gmail.com>

allow --bind-addr (not announce) to work with tor autoservice

fix issue reported by @gabridome

Signed-off-by: Saibato <Saibato.naga@pm.me>

allow --bind-addr (not announce) to work with tor autoservice

fix issue ElementsProject#2183 as reported by @gabridome
  • Loading branch information
cdecker authored and Saibato committed Dec 19, 2018
1 parent 05dc095 commit 705fe47
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions connectd/connectd.c
Original file line number Diff line number Diff line change
Expand Up @@ -1060,9 +1060,6 @@ static struct wireaddr_internal *setup_listeners(const tal_t *ctx,
if (!(proposed_listen_announce[i] & ADDR_LISTEN))
continue;

if (!(proposed_listen_announce[i] & ADDR_ANNOUNCE))
continue;

if (proposed_wireaddr[i].itype != ADDR_INTERNAL_AUTOTOR)
continue;

Expand Down
2 changes: 1 addition & 1 deletion lightningd/plugin.c
Original file line number Diff line number Diff line change
Expand Up @@ -848,7 +848,7 @@ char *add_plugin_dir(struct plugins *plugins, const char *dir, bool nonexist_ok)
if (!d) {
if (nonexist_ok && errno == ENOENT)
return NULL;
return tal_fmt("Failed to open plugin-dir %s: %s",
return tal_fmt(NULL, "Failed to open plugin-dir %s: %s",
dir, strerror(errno));
}

Expand Down

0 comments on commit 705fe47

Please sign in to comment.