Skip to content

Improve wording and typography in May 13 advisories#2877

Open
kotauskas wants to merge 10 commits into
rustsec:mainfrom
kotauskas:may-13-fixes
Open

Improve wording and typography in May 13 advisories#2877
kotauskas wants to merge 10 commits into
rustsec:mainfrom
kotauskas:may-13-fixes

Conversation

@kotauskas
Copy link
Copy Markdown
Contributor

This PR includes the following improvements to several of the advisories that were assigned numbers today, as well as all of Diesel's 2026 vulnerabilities:

  • Improved wording
  • Fixed typographical inconsistencies
  • Added some missing metadata (most prominently [affected.functions])
  • A few clarifications and added context to help readers accurately judge the impact of an advisory

Additionally, RUSTSEC-2026-0133 has had its "informational" label removed, as it is a guaranteed out-of-bounds access that affects all users of the crate, not requiring it to be used in some unintended or buggy way.

@kotauskas
Copy link
Copy Markdown
Contributor Author

Not sure what GitHub is trying to tell me here.
Empty error message in CI

@Skgland
Copy link
Copy Markdown
Contributor

Skgland commented May 13, 2026

Not sure what GitHub is trying to tell me here. Empty error message in CI

For me it look like this:
image

The error is because the keys of entries in [affected.functions] are not allowed to contain spaces.
Also the [affected.functions] entries for diesel-async won't work as the affected functions are in a different crate.

@kotauskas
Copy link
Copy Markdown
Contributor Author

Fixed, thanks for the advice. It turns out that the broken output in GitHub Actions was due to NoScript blocking windows.net on my end, which GitHub previously wasn't making any requests to (what the hell Microsoft).

@djc
Copy link
Copy Markdown
Member

djc commented May 14, 2026

Please squash the changes from your last commit back into the originating commits.

Copy link
Copy Markdown
Member

@djc djc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the careful review.

Some of these changes err on the side of being overly pedantic to me, and in the future I don't think submitting PRs to fix capitalization or reflow lines are worth my review time. On the other hand, adding the affected functions and improving (or removing) the informational status is definitely valuable.

Do you want to help improve

to cover some of this stuff?

# Unsound access to padding bytes while deserializing date/time values using the MySQL backend

This affects any usage of the following functions with a `AsyncMysqlConnection` provided by diesel-async:
`diesel-async` uses the `mysql-async` crate for interacting with
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not reflow in the same commit as changing wording, as that makes it very hard to review.


In libcrux-ml-dsa, hint decoding did not check the boundedness of the
cumulative hint counter of the last row of the hint vector.
In `libcrux-ml-dsa`, hint decoding did not check the boundedness of
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think quoting crate names makes sense.

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.

3 participants