GHSA-q669-2vfg-cxcg
Nervos CKB Unaligned Pointer Dereference
- Severity
- Medium
- The publisher's own rating, not a CVSS score
- Exploit chance (EPSS)
- —
- No score published
- Known exploited
- No
- Not in CISA's catalog
- Weakness
- —
- Not classified
- Fix
0.31.1- Upgrade ckb to this version or later
Description
There are multiple type conversions in ckb that unsafely cast between byte pointers and other types of pointers. This results in unaligned pointers, which are not allowed by the Rust language, and are considered undefined behavior, meaning that the compiler is free to do anything with code. This can lead to unpredictable bugs that can become security vulnerabilities.
Some of the bugs here could potentially lead to buffer overreads in malformed data (it's not clear to me as I haven't investigated the practical impact of these bugs).
Two of these (in blockchain.rs) do not create unaligned data. They do though perform an unsafe operation that may not uphold the invariants of the safe function they are in, and could lead to undefined behavior and buffer overreads on malformed input.
These are of the same nature as those in my previous report about the molecule crate.
Patch attached for commit 1b09e37c8e1b7945495cd18d9782417fbe51e986 that fixes all cases I know of at this time.
Please consider this report for reward under the terms of the bug bounty program.
Related advisory: https://github.com/nervosnetwork/molecule/security/advisories/GHSA-rffv-8x7x-p7pw
Affected versions
crates.io
| Package | Vulnerable | Fixed in |
|---|---|---|
| ckb | < 0.31.1 |
0.31.1 |
How to fix
Move to the fixed version or later with your package manager, update the lockfile, and run your tests. If the package only arrives through another dependency, upgrade that dependency first. If none of its releases pulls a fixed version yet, force the version with your package manager's override.
ckb crates.io → 0.31.1
- cargo
cargo update -p ckb --precise 0.31.1
References
Other links
Timeline
- Published
Related advisories
| Advisory | Severity | Why it's related |
|---|---|---|
| GHSA-29c2-65rj-h343 Nervos CKB Permit load cell data from memory | Medium | Also affects ckb |
| GHSA-h4c3-5275-vrmg Nervos CKB Pool does not remove the conflicting transactions from the statistics | Medium | Also affects ckb |
| GHSA-q73f-w3h7-7wcc Nervos CKB Transaction which calls syscall load_cell_data_hash has nondeterministic result | Critical | Also affects ckb |
| GHSA-3gjh-29fv-8hr6 Nervos CKB Snappy decompress length can be very large and causes out of memory error | High | Also affects ckb |
| GHSA-wjxc-pjx9-4wvm Nervos CKB Panic on malformed input | High | Also affects ckb |
| GHSA-hjqq-29pw-96wj Nervos CKB node panics when processing a block which parent timestamp is too new | High | Also affects ckb |
| GHSA-r9rv-9mh8-pxf4 Nervos CKB BlockTimeTooNew should not be considered as invalid block | Medium | Also affects ckb |
| GHSA-pr39-8257-fxc2 Nervos CKB DoS: Process exists when p2p discovery protocol receives unsupported peer IP | Low | Also affects ckb |
MergeHarbor checks every repository's dependencies against this database, ranks what matters by exploitability and reachability, routes it to the owning team and opens the fix pull request.
Check your repos with MergeHarbor