GHSA-4g52-pqcj-phvh
Suggest an improvement- Source
- https://github.com/advisories/GHSA-4g52-pqcj-phvh
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2021/05/GHSA-4g52-pqcj-phvh/GHSA-4g52-pqcj-phvh.json
- JSON Data
- https://api.test.osv.dev/v1/vulns/GHSA-4g52-pqcj-phvh
- Aliases
- Related
- Published
- 2021-05-21T16:23:43Z
- Modified
- 2024-08-21T15:57:17.987831Z
- Severity
-
- 5.9 (Medium) CVSS_V3 - CVSS:3.1/AV:N/AC:H/PR:N/UI:N/S:U/C:N/I:H/A:N CVSS Calculator
- Summary
- BLS Signature "Malleability"
- Details
-
Impact
- BLS signature validation in lotus uses blst library method VerifyCompressed. This method accepts signatures in 2 forms - "serialized", and "compressed", meaning that BLS signatures can be provided as either of 2 unique byte arrays.
- Lotus block validation functions perform a uniqueness check on provided blocks. Two blocks are considered distinct if the CIDs of their blockheader do not match. The CID method for blockheader includes the BlockSig of the block.
The result of these issues is that it would be possible to punish miners for valid blocks, as there are two different valid block CIDs available for each block, even though this must be unique.
Patches
By switching from the go based
blstbindings over to the bindings infilecoin-ffi, the code paths now ensure that all signatures are compressed by size and the way they are deserialized. This happened in https://github.com/filecoin-project/lotus/pull/5393References
- Original POC: https://gist.github.com/wadeAlexC/2490d522e81a796af9efcad1686e6754
- Database specific
{ "nvd_published_at": "2021-04-15T22:15:00Z", "github_reviewed_at": "2021-05-20T20:51:00Z", "severity": "MODERATE", "github_reviewed": true, "cwe_ids": [ "CWE-347" ] }- References
Affected packages
Go / github.com/filecoin-project/lotus
Package
- Name
- github.com/filecoin-project/lotus
- View open source insights on deps.dev
- Purl
- pkg:golang/github.com/filecoin-project/lotus