Skip to content

Add advisory for bitm RankSimple unsoundness#2997

Draft
sisy2020 wants to merge 1 commit into
rustsec:mainfrom
sisy2020:add-bitm-ranksimple-advisory
Draft

Add advisory for bitm RankSimple unsoundness#2997
sisy2020 wants to merge 1 commit into
rustsec:mainfrom
sisy2020:add-bitm-ranksimple-advisory

Conversation

@sisy2020

Copy link
Copy Markdown
Contributor

Affected crate(s)

  • bitm (129557 recent downloads per crates.io)

Links to upstream issue(s) or PR(s)

Severity

Soundness issue in a safe public API. In affected versions, safe code could violate RankSimple's internal invariants and cause try_rank to perform an out-of-bounds get_unchecked read, which is undefined behavior.

The issue has been fixed in 0.5.2 by making the RankSimple fields private.

Checklist

  • Advisory filename(s) starts with RUSTSEC-0000-0000 as the ID
  • date field is set to the public disclosure date
  • Contains a concise and descriptive title after advisory metadata
  • Asked maintainer(s) if publishing an advisory is appropriate

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