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

Enable MemorySanitizer in CI tests #17

Closed
wants to merge 1 commit into from
Closed

Conversation

intgr
Copy link
Owner

@intgr intgr commented Feb 20, 2020

No description provided.

@intgr
Copy link
Owner Author

intgr commented Feb 20, 2020

  • Need to investigate MemorySanitizer output:
Uninitialized bytes in __interceptor_memchr at offset 0 inside [0x701000000000, 4)
==50==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x56223dbdf77e  (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0x9c77e)
    #1 0x56223dbe7cf1  (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0xa4cf1)
    #2 0x56223dbd04eb  (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0x8d4eb)
    #3 0x56223dbb5cc1  (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0x72cc1)
    #4 0x7f8b3ca1409a  (/lib/x86_64-linux-gnu/libc.so.6+0x2409a)
    #5 0x56223db4f1d9  (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0xc1d9)

SUMMARY: MemorySanitizer: use-of-uninitialized-value (/root/build/target/debug/build/libc-e05582269760f0a4/build-script-build+0x9c77e) 

@intgr intgr force-pushed the enable-memorysanitizer branch from d03a78f to 1eddbda Compare March 2, 2020 23:57
@intgr
Copy link
Owner Author

intgr commented Mar 3, 2020

Issue very likely related to rust-lang/rust#39610

The good news is that the stack trace actually comes from libc's build-script-build script, not caused by posix-acl, and probably false positive. :)

@intgr
Copy link
Owner Author

intgr commented Mar 3, 2020

Memory Sanitizer is extra tricky and also requires you to rebuild ALL linked code with it, including the standard library. See https://doc.rust-lang.org/nightly/unstable-book/compiler-flags/sanitizer.html for details

I'd say this is not worth it, rebuilding the whole standard library would make the CI take unreasonably long. Closing for now.

@intgr intgr closed this Mar 3, 2020
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