Skip to content

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Aug 22, 2025

The fuzz/ and record/ directories are about 2MiB, and are not used by the end user. This removes them from the crate for future publishes.

Also, this avoids risk of an xz-style attack leveraging the binary files produced by fuzzing, where malicious code could be slipped in past review through binary test data.

The fuzz/ and record/ directories are about 2MiB, and are not used by
the end user. This removes them from the crate for future publishes.

Also, this avoids risk of an [xz-style attack] leveraging the binary
files produced by fuzzing, where malicious code could be slipped in past
review through binary test data.

[zx-style attack]: https://en.wikipedia.org/wiki/XZ_Utils_backdoor
Copy link
Member

@BurntSushi BurntSushi left a comment

Choose a reason for hiding this comment

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

Thanks!

@BurntSushi BurntSushi merged commit 71233b5 into rust-lang:master Aug 23, 2025
16 checks passed
erickt added a commit to erickt/regex that referenced this pull request Aug 25, 2025
I missed more binary test data `tests/fuzz/` in rust-lang#1281, which should
remove another 56KiB from the regex crate.
BurntSushi pushed a commit that referenced this pull request Aug 25, 2025
I missed more binary test data `tests/fuzz/` in #1281, which should
remove another 56KiB from the regex crate.

PR #1289
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.

2 participants