3 Commits

Author	SHA1	Message	Date
Rusty Russell	c919551109	onchaind: include htlc id in htlc_stub so we agree on what HTLC we're closing. ... If there are two HTLCs with the same preimage, lightningd would always find the first one. By including the id in the `struct htlc_stub` it's both faster (normal HTLC lookup) and allows lightningd to detect that onchaind wants to fail both of them. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>	2018-10-23 16:55:35 +02:00
Rusty Russell	fed5a117e7	Update ccan/structeq. ... structeq() is too dangerous: if a structure has padding, it can fail silently. The new ccan/structeq instead provides a macro to define foo_eq(), which does the right thing in case of padding (which none of our structures currently have anyway). Upgrade ccan, and use it everywhere. Except run-peer-wire.c, which is only testing code and can use raw memcmp(): valgrind will tell us if padding exists. Interestingly, we still declared short_channel_id_eq, even though we didn't define it any more! Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>	2018-07-04 23:57:00 +02:00
Rusty Russell	bbed5e3411	Rename subdaemons, move them into top level. ... We leave the *build* results in lightningd/ for ease of in-place testing though. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>	2017-08-29 17:54:14 +02:00