Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add verification when constructing the TrustedConsensusState #369

Closed
4 tasks
srdtrk opened this issue Mar 6, 2025 · 0 comments · Fixed by #383
Closed
4 tasks

Add verification when constructing the TrustedConsensusState #369

srdtrk opened this issue Mar 6, 2025 · 0 comments · Fixed by #383
Assignees
Labels
audit bug Something isn't working ethereum-light-client Issues related to the 08-wasm ethereum light client

Comments

@srdtrk
Copy link
Member

srdtrk commented Mar 6, 2025

Summary of Bug

Internal audit item: When we construct the TrustedConsensusState using the stored consensus state + the relayer provided sync committee. We need to validate that the relayer provided sync committee matches the aggregated key stored in the consensus state

Expected Behaviour

TrustedConsensusState is constructed via a constructor, and that constructor needs to validate:

  • Aggregate key that the relayer provides matches the full sync committee that the relayer provides
  • Aggregate key that the relayer provides matches the stored aggregate key

Version

main


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
  • Estimate provided
@srdtrk srdtrk added bug Something isn't working ethereum-light-client Issues related to the 08-wasm ethereum light client audit labels Mar 6, 2025
@github-project-automation github-project-automation bot moved this to Backlog in IBC Mar 8, 2025
@srdtrk srdtrk moved this from Backlog to In progress in IBC Mar 10, 2025
@srdtrk srdtrk self-assigned this Mar 10, 2025
@github-project-automation github-project-automation bot moved this from In progress to Done in IBC Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit bug Something isn't working ethereum-light-client Issues related to the 08-wasm ethereum light client
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant