Skip to content

smite: add funding_signed oracle - #212

Open
NishantBansal2003 wants to merge 1 commit into
lnfuzz:masterfrom
NishantBansal2003:funding-signed-oracle
Open

NishantBansal2003 wants to merge 1 commit into
lnfuzz:masterfrom
NishantBansal2003:funding-signed-oracle

Conversation

@NishantBansal2003

Copy link
Copy Markdown
Contributor

Add FundingSignedOracle and consolidate all the previously scattered checks relevant to the funding flow negotiation into it, including unknown channel_id, cases where the target accepted an invalid funding_created we sent, and cases where the target sent an invalid funding_signed.

@morehouse morehouse left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs rebase.

Comment thread smite-scenarios/src/executor.rs Outdated
}

#[test]
fn execute_recv_channel_ready_invalid_signature_is_noop() {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also needs a test that RecvFundingSigned reports a violation when the funding_created signature is invalid.

Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants