Skip to content

Tracking issue for reviewing and moving sanitizers into a compiler crate (i.e., rustc_sanitizers) #123619

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
2 of 4 tasks
rcvalle opened this issue Apr 8, 2024 · 2 comments
Open
2 of 4 tasks
Labels
A-sanitizers Area: Sanitizers for correctness and code quality C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC PG-exploit-mitigations Project group: Exploit mitigations

Comments

@rcvalle
Copy link
Member

rcvalle commented Apr 8, 2024

This is a tracking issue for reviewing previously-added support for sanitizers to the Rust compiler, and moving and organizing as much as possible of the sanitizers source code into a compiler crate (i.e., rustc_sanitizers). This is part of our work to organize and stabilize support for the sanitizers. (See our roadmap at https://hackmd.io/@rcvalle/S1Ou9K6H6.)

Steps

  • Create the rustc_sanitizers compiler crate.
  • Move the source code for the CFI and KCFI sanitizers to the rustc_sanitizers crate.
  • Move common sanitizers source code, review and move other sanitizers to the rustc_sanitizers crate.
  • TBD.

Unresolved Questions

None.

Implementation history

@rcvalle rcvalle added C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC PG-exploit-mitigations Project group: Exploit mitigations labels Apr 8, 2024
@Jules-Bertholet
Copy link
Contributor

@rustbot label A-sanitizers

@rustbot rustbot added the A-sanitizers Area: Sanitizers for correctness and code quality label Apr 8, 2024
@rcvalle rcvalle changed the title This is a tracking issue for reviewing and moving sanitizers into a compiler crate (i.e., rustc_sanitizers) Tracking issue for reviewing and moving sanitizers into a compiler crate (i.e., rustc_sanitizers) Apr 8, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Apr 9, 2024
…rs, r=davidtwco

sanitizers: Create the rustc_sanitizers crate

Create the `rustc_sanitizers` crate and move the source code for the CFI and KCFI sanitizers to it. The tracking issue for reviewing and moving sanitizers into a compiler crate is rust-lang#123619. This is part of our work to organize and stabilize support for the sanitizers. (See our roadmap at https://hackmd.io/`@rcvalle/S1Ou9K6H6.)`
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Apr 9, 2024
Rollup merge of rust-lang#123620 - rcvalle:rust-create-rustc-sanitizers, r=davidtwco

sanitizers: Create the rustc_sanitizers crate

Create the `rustc_sanitizers` crate and move the source code for the CFI and KCFI sanitizers to it. The tracking issue for reviewing and moving sanitizers into a compiler crate is rust-lang#123619. This is part of our work to organize and stabilize support for the sanitizers. (See our roadmap at https://hackmd.io/`@rcvalle/S1Ou9K6H6.)`
@workingjubilee
Copy link
Member

@rcvalle are all the sanitizers that closely related? I was under the impression that sanitizers of the ASan nature and the CFI sanitizers were fairly different.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-sanitizers Area: Sanitizers for correctness and code quality C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC PG-exploit-mitigations Project group: Exploit mitigations
Projects
None yet
Development

No branches or pull requests

4 participants