-
Notifications
You must be signed in to change notification settings - Fork 373
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add advisory for nano-id crate (#1974)
* Add advisory for nano-id * Update advisory for nano-id
- Loading branch information
Showing
1 changed file
with
73 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,73 @@ | ||
```toml | ||
[advisory] | ||
id = "RUSTSEC-0000-0000" | ||
package = "nano-id" | ||
date = "2024-06-03" | ||
categories = ["crypto-failure"] | ||
cvss = "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:L" | ||
|
||
[affected] | ||
functions = { "nano_id::base58" = ["< 0.4.0"], "nano_id::base62" = ["< 0.4.0"], "nano_id::gen" = ["< 0.4.0"] } | ||
|
||
[versions] | ||
patched = [">= 0.4.0"] | ||
``` | ||
|
||
# Reduced entropy due to inadequate character set usage | ||
|
||
## Description | ||
|
||
Affected versions of the nano-id crate incorrectly generated IDs using a reduced character set in the `nano_id::base62` and `nano_id::base58` functions. Specifically, the `base62` function used a character set of 32 symbols instead of the intended 62 symbols, and the `base58` function used a character set of 16 symbols instead of the intended 58 symbols. Additionally, the `nano_id::gen` macro is also affected when a custom character set that is not a power of 2 in size is specified. | ||
|
||
It should be noted that `nano_id::base64` is not affected by this vulnerability. | ||
|
||
## Impact | ||
|
||
This can result in a significant reduction in entropy, making the generated IDs predictable and vulnerable to brute-force attacks when the IDs are used in security-sensitive contexts such as session tokens or unique identifiers. | ||
|
||
## Patches | ||
|
||
The flaws were corrected in commit [a9022772b2f1ce38929b5b81eccc670ac9d3ab23](https://github.com/viz-rs/nano-id/commit/a9022772b2f1ce38929b5b81eccc670ac9d3ab23) by updating the the `nano_id::gen` macro to use all specified characters correctly. | ||
|
||
## PoC | ||
|
||
```rust | ||
use std::collections::BTreeSet; | ||
|
||
fn main() { | ||
test_base58(); | ||
test_base62(); | ||
} | ||
|
||
fn test_base58() { | ||
let mut produced_symbols = BTreeSet::new(); | ||
|
||
for _ in 0..100_000 { | ||
let id = nano_id::base58::<10>(); | ||
for c in id.chars() { | ||
produced_symbols.insert(c); | ||
} | ||
} | ||
|
||
println!( | ||
"{} symbols generated from nano_id::base58", | ||
produced_symbols.len() | ||
); | ||
} | ||
|
||
fn test_base62() { | ||
let mut produced_symbols = BTreeSet::new(); | ||
|
||
for _ in 0..100_000 { | ||
let id = nano_id::base62::<10>(); | ||
for c in id.chars() { | ||
produced_symbols.insert(c); | ||
} | ||
} | ||
|
||
println!( | ||
"{} symbols generated from nano_id::base62", | ||
produced_symbols.len() | ||
); | ||
} | ||
``` |