Skip to content

smite: extend accept_channel oracle with sanity and key reuse checks - #259

Open
NishantBansal2003 wants to merge 5 commits into
lnfuzz:masterfrom
NishantBansal2003:more-accept-chan-oracles
Open

NishantBansal2003 wants to merge 5 commits into
lnfuzz:masterfrom
NishantBansal2003:more-accept-chan-oracles

Conversation

@NishantBansal2003

Copy link
Copy Markdown
Contributor

ref: #209 (comment)

Some more oracles added in this PR:

open_channel

  • chain_hash != regtest
  • channel_reserve_satoshis >= funding_satoshis
  • dust_limit_satoshis > 10_000 sat
  • to_self_delay > 2016
  • feerate_per_kw < 253 for non 0FC channels

accept_channel

  • dust_limit_satoshis > 10_000 sat
  • htlc_minimum_msat > max_htlc_value_in_flight_msat
  • htlc_minimum_msat > open_channel.funding_satoshis
  • to_self_delay < 144
  • max_accepted_htlcs == 0
  • pubkey match any pubkeys from open_channel and accept_channel
  • per_commitment_point should not be reused

Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>
Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>
Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>
Signed-off-by: Nishant Bansal <nishant.bansal.282003@gmail.com>
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.

1 participant