Skip to content

fix: guard nil light client attack signed header#59

Draft
UnbornAztecKing wants to merge 1 commit into
dydx-fork-v0.38.5from
cantina-133-signed-header-guard
Draft

fix: guard nil light client attack signed header#59
UnbornAztecKing wants to merge 1 commit into
dydx-fork-v0.38.5from
cantina-133-signed-header-guard

Conversation

@UnbornAztecKing

Copy link
Copy Markdown

Summary

  • Add a nil SignedHeader guard before LightClientAttackEvidence accesses the promoted Header field.
  • Add direct validation and protobuf decode regressions for omitted conflicting_block.signed_header.

Validation

  • go test ./types -run TestLightClientAttackEvidenceValidation
  • go test ./types -run 'TestLightClientAttackEvidence|TestEvidenceFromProto'
  • go test ./types
  • go test ./consensus ./state ./types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant