Skip to content

fix: panic when loading CRS with duplicate points from bytes#105

Open
timimm wants to merge 1 commit intocrate-crypto:masterfrom
timimm:master
Open

fix: panic when loading CRS with duplicate points from bytes#105
timimm wants to merge 1 commit intocrate-crypto:masterfrom
timimm:master

Conversation

@timimm
Copy link

@timimm timimm commented Nov 14, 2025

No description provided.

tx-tomcat pushed a commit to Loragon-chain/rust-verkle that referenced this pull request Dec 21, 2025
- Add CRSError enum with InvalidLength, DuplicatePoints, and
  PointDeserializationError variants
- Add try_from_bytes_uncompressed method to Element for safe deserialization
- Update CRS::from_bytes to return Result<CRS, CRSError>
- Add validate_no_duplicates helper for duplicate point detection
- Add deprecated from_bytes_unchecked wrapper for backward compatibility
- Add comprehensive unit tests for all error paths
- Add thiserror dependency to ipa-multipoint

Closes part of PR crate-crypto#105 - fixes panic on invalid CRS bytes

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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