Skip to content

fix: bound basic constraints path length - #77

Merged
leopoldjoy merged 3 commits into
mainfrom
audit/bound-pathlen-int64
Jul 31, 2026
Merged

fix: bound basic constraints path length#77
leopoldjoy merged 3 commits into
mainfrom
audit/bound-pathlen-int64

Conversation

@leopoldjoy

Copy link
Copy Markdown
Contributor

Summary

  • reject BasicConstraints pathLenConstraint values above int64.max before casting
  • preserve canonical DER INTEGERs within the contract range without imposing a fixed byte-length cap
  • add boundary tests for int64.max and 2^63

Verification

  • forge test --match-contract CertManagerTest
  • forge build --sizes (compile succeeds; raw size command still fails on oversized test harnesses, addressed by prerequisite ci: scope size check to deployables #73; deployable CertManager margin is 75 bytes)

leopoldjoy and others added 2 commits July 27, 2026 14:35
Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@leopoldjoy
leopoldjoy merged commit e6382f8 into main Jul 31, 2026
8 checks passed
@leopoldjoy
leopoldjoy deleted the audit/bound-pathlen-int64 branch July 31, 2026 15:48
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