[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 Yury Gribov y.gribov at samsung dot com changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #8 from Yury Gribov y.gribov at samsung dot com --- Works for me in Clang and GCC trunks.
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718
--- Comment #5 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Alexey Samsonov from comment #2)
I've already fixed the problem with crashes caused by small
malloc_context_size values. I think you should wait till the ASan runtime
will be merged from LLVM into GCC.
I can reproduce this with current Clang/compiler-rt:
$ cat main.cpp
int main() {
delete new char[20];
return 0;
}
$ ~/install/bin/x86_64-unknown-linux-gnu-clang -g -O0 main.cpp
-fsanitize=address -o a.out
$ ASAN_OPTIONS=malloc_context_size=0:fast_unwind_on_malloc=0 ./a.out
...
==15804==AddressSanitizer CHECK failed:
/mnt/scratch/ygribov/llvm/projects/compiler-rt/lib/asan/asan_allocator2.cc:237
((id)) != (0) (0x0, 0x0)
-Y
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 --- Comment #6 from Kostya Serebryany kcc at gcc dot gnu.org --- Can we keep this bug in one place, please? Let https://code.google.com/p/address-sanitizer/issues/detail?id=239 be the primary one
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 --- Comment #7 from Yuri Gribov tetra2005 at gmail dot com --- (In reply to Kostya Serebryany from comment #6) Can we keep this bug in one place, please? Let https://code.google.com/p/address-sanitizer/issues/detail?id=239 be the primary one Ok, will do. I'm a little gcc-biased as you know.
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 --- Comment #1 from Yury Gribov y.gribov at samsung dot com --- Created attachment 30999 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=30999action=edit Proposed patch It seems that this CHECK should be removed to allow disabling malloc tracing. Patch is attached. -Y
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 Alexey Samsonov samsonov at google dot com changed: What|Removed |Added CC||samsonov at google dot com --- Comment #2 from Alexey Samsonov samsonov at google dot com --- I've already fixed the problem with crashes caused by small malloc_context_size values. I think you should wait till the ASan runtime will be merged from LLVM into GCC.
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 --- Comment #3 from Yury Gribov y.gribov at samsung dot com --- Got it. I wonder whether we should keep the bug opened until we merge or close it now.
[Bug sanitizer/58718] Invalid check in libsanitizer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58718 --- Comment #4 from Kostya Serebryany kcc at gcc dot gnu.org --- I am currently testing a merge which is before Alexey's changes. There is no harm in keeping this bug open.
