vincenzopalazzo
/
rgb-cln
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

channeld: disable check for my_current_per_commitment_point. 

Under stress, it fails (test_restart_many_payments, the next test).

I suspect a deep misunderstanding in the comparison code, will chase
separately.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is contained in:
Rusty Russell 2018-10-09 19:22:52 +10:30
parent ec74aadce2
commit 1526f024fb
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
channeld/channeld.c
View File

@ -1940,6 +1940,7 @@ static void check_current_dataloss_fields(struct peer *peer,
type_to_string(tmpctx, struct secret,
&old_commit_secret));
#if 0 /* FIXME: This isn't reliable! */
/* FIXME: We don't keep really old per_commit numbers, so we can't
* check this 'needs retransmit' case! */
if (next_remote_revocation_number == peer->next_index[REMOTE]) {
@ -1963,6 +1964,7 @@ static void check_current_dataloss_fields(struct peer *peer,
remote_current_per_commitment_point),
next_remote_revocation_number,
peer->next_index[REMOTE]);
#endif
status_trace("option_data_loss_protect: fields are correct");
}