Skip to content

Commit

Permalink
build: fix compile error on gcc ~7.3.0
Browse files Browse the repository at this point in the history
It seems to be having a bit of trouble understanding the control flow to realize
it's not actually uninitialized.

Add an error handler after the switch in case we miss a real uninitialized error
in the future.

Signed-off-by: William Casarin <[email protected]>
  • Loading branch information
jb55 authored and rustyrussell committed Oct 9, 2018
1 parent 45b07e7 commit 7c4b9c8
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions devtools/gossipwith.c
Original file line number Diff line number Diff line change
Expand Up @@ -150,11 +150,11 @@ int main(int argc, char *argv[])
{
struct io_conn *conn = tal(NULL, struct io_conn);
struct wireaddr_internal addr;
int af;
int af = -1;
struct pubkey us, them;
const char *err_msg;
const char *at;
struct addrinfo *ai;
struct addrinfo *ai = NULL;

setup_locale();
secp256k1_ctx = secp256k1_context_create(SECP256K1_CONTEXT_VERIFY |
Expand Down Expand Up @@ -212,6 +212,10 @@ int main(int argc, char *argv[])
}
ai = wireaddr_to_addrinfo(tmpctx, &addr.u.wireaddr);
}

if (af == -1 || ai == NULL)
err(1, "Initializing socket");

conn->fd = socket(af, SOCK_STREAM, 0);
if (conn->fd < 0)
err(1, "Creating socket");
Expand Down

0 comments on commit 7c4b9c8

Please sign in to comment.