Skip to content
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

[Ubuntu Upgrade][docs] Say need to build msan deps manually #6238

Merged
merged 2 commits into from
Aug 20, 2021

Conversation

jonathanmetzman
Copy link
Contributor

All dependencies must be built with MSAN by the project.
OSS-Fuzz will not be providing any instrumented libraries.

Related: #6180

All dependencies must be built with MSAN by the project.
OSS-Fuzz will not be providing any instrumented libraries.

Related: #6180
@jonathanmetzman jonathanmetzman changed the title [Ubuntu Upgrade] Upgrade msan parts to reflect new reality. [Ubuntu Upgrade][docs] Upgrade msan parts to reflect new reality. Aug 17, 2021
@jonathanmetzman jonathanmetzman changed the title [Ubuntu Upgrade][docs] Upgrade msan parts to reflect new reality. [Ubuntu Upgrade][docs] Upgrade msan bits to say need to build deps with msan manually Aug 17, 2021
@jonathanmetzman jonathanmetzman changed the title [Ubuntu Upgrade][docs] Upgrade msan bits to say need to build deps with msan manually [Ubuntu Upgrade][docs] Change Msan bits to say need to build deps with msan manually Aug 17, 2021
@jonathanmetzman jonathanmetzman changed the title [Ubuntu Upgrade][docs] Change Msan bits to say need to build deps with msan manually [Ubuntu Upgrade][docs] Say need to build msan deps manually Aug 17, 2021
docs/getting-started/new_project_guide.md Outdated Show resolved Hide resolved
[msan-libs-builder Dockerfile](https://github.com/google/oss-fuzz/blob/master/infra/base-images/msan-libs-builder/Dockerfile#L20).
un-instrumented system dependencies.
If you want to use "memory," please build all libraries your project needs using
MemorySanitizer.
Copy link
Collaborator

Choose a reason for hiding this comment

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

mention "and statically link them into the binary"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why? We do support dynamic linking, I suspect this would work with msan too.
Maybe I should just omit this part you are suggesting. WDYT?

Change-Id: I5fffa344ebc1c696a3626442d8735efbefff0ef9
@jonathanmetzman jonathanmetzman marked this pull request as ready for review August 20, 2021 01:18
@jonathanmetzman jonathanmetzman merged commit 6bcbae3 into master Aug 20, 2021
@jonathanmetzman jonathanmetzman deleted the msan-docs branch August 20, 2021 03:06
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