New issue
Advanced search Search tips

Issue 695660 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: ----
Type: Bug



Sign in to add a comment

MSan reports in libFuzzer engine.

Project Member Reported by ClusterFuzz, Feb 23 2017

Issue description

Comment 1 by och...@chromium.org, Feb 23 2017

Labels: -Type-Bug-Security -Restrict-View-SecurityTeam -Security_Severity-Medium -Unreproducible Type-Bug
Owner: kcc@chromium.org
Status: Assigned (was: Untriaged)
Summary: MSan reports in libFuzzer engine. (was: Use-of-uninitialized-value in _start)
Kostya, mind taking a look at this? Has this already been fixed upstream?

Comment 4 by aarya@google.com, Feb 24 2017

Cc: kcc@chromium.org
Owner: ----
Ok rolling forward.

Comment 5 by kcc@chromium.org, Feb 24 2017

Confirmed. I can reproduce the error and with the updated libFuzzer -- I can't
Cc: -kcc@chromium.org
Owner: kcc@chromium.org
The 2 bugs are still there (https://cluster-fuzz.appspot.com/v2/crash-stats), now showing up in memmem

https://cluster-fuzz.appspot.com/v2/testcase-detail/4537313750417408
Uninitialized bytes in __interceptor_memmem at offset 15 inside [0x7ffe69bc9cd1, 40)
==24543==WARNING: MemorySanitizer: use-of-uninitialized-value
#0 0x512dd9 in fuzzer::SearchMemory(void const*, unsigned long, void const*, unsigned long) third_party/libFuzzer/src/FuzzerUtilPosix.cpp:118:10
#1 0x4ef769 in fuzzer::MutationDispatcher::MakeDictionaryEntryFromCMP(void const*, void const*, void const*, void const*, unsigned long, unsigned char const*, unsigned long) third_party/libFuzzer/src/FuzzerMutate.cpp:220:24
#2 0x4eff01 in fuzzer::MutationDispatcher::MakeDictionaryEntryFromCMP(fuzzer::FixedWord<64ul> const&, fuzzer::FixedWord<64ul> const&, unsigned char const*, unsigned long) third_party/libFuzzer/src/FuzzerMutate.cpp:245:10
#3 0x4ecfd8 in fuzzer::MutationDispatcher::Mutate_AddWordFromTORC(unsigned char*, unsigned long, unsigned long) third_party/libFuzzer/src/FuzzerMutate.cpp:267:10
#4 0x4f4eed in fuzzer::MutationDispatcher::MutateImpl(unsigned char*, unsigned long, unsigned long, std::__1::vector<fuzzer::MutationDispatcher::Mutator, std::__1::allocator<fuzzer::MutationDispatcher::Mutator> > const&) third_party/libFuzzer/src/FuzzerMutate.cpp:518:22
#5 0x4c8877 in fuzzer::Fuzzer::MutateAndTestOne() third_party/libFuzzer/src/FuzzerLoop.cpp:750:18
#6 0x4c9481 in fuzzer::Fuzzer::Loop() third_party/libFuzzer/src/FuzzerLoop.cpp:791:5
#7 0x49b337 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) third_party/libFuzzer/src/FuzzerDriver.cpp:567:6
#8 0x4d3ed0 in main third_party/libFuzzer/src/FuzzerMain.cpp:20:10
#9 0x7f70c4318f44 in __libc_start_main /build/eglibc-oGUzwX/eglibc-2.19/csu/libc-start.c:287
#10 0x422f64 in _start


Project Member

Comment 8 by ClusterFuzz, Feb 25 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase 6217522404917248 is verified as fixed, so closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Comment 9 by kcc@chromium.org, Feb 28 2017

Cc: infe...@chromium.org
inferno, not sure what the status is? 
This bug got auto-closed? Should we re-open? 
Labels: -ClusterFuzz-Verified ClusterFuzz-Wrong
Status: Assigned (was: Verified)
Yes it still exists, attaching the new stacks.

Comment 12 by kcc@chromium.org, Feb 28 2017

Cc: kcc@chromium.org
Owner: aizatsky@chromium.org
I may not have time in 1-2 weeks :( 
assigning to Mike

Comment 13 by ta...@google.com, Mar 23 2017

Owner: kcc@chromium.org
Also seeing this on OSS-Fuzz side: https://github.com/google/oss-fuzz/issues/473#issuecomment-288497760

Comment 15 by kcc@chromium.org, Mar 23 2017

ACK. 
We need to do some reconstructive surgery in the upstream LLVM build & test system -- currently we don't run libFuzzer's own tests with msan so we don't see these. (Sadly, the change won't be simple, and I may not have cycles for it).
Meanwhile I'll try to fix the crashes you observe on oss-fuzz & CF

Comment 16 by kcc@chromium.org, Mar 23 2017

I think the one in #14 should be fixed now. 
Thanks! Kicked off new builds for OSS-Fuzz and rolling libFuzzer (https://codereview.chromium.org/2775633003/). 
Status: WontFix (was: Assigned)
Bulk-WontFixing these bugs. This was a bug on ClusterFuzz side, see bug 717534. We will start seeing new testcases auto-filed in a day or two. We can't leave these open as ClusterFuzz won't autoverify them after ClusterFuzz-Wrong label.
Labels: -ClusterFuzz-Wrong
We have made a bunch of changes on ClusterFuzz side, so resetting ClusterFuzz-Wrong label.

Sign in to add a comment