New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 705255 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 10
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug

Blocking:
issue 703833



Sign in to add a comment

Regression: NativeLibraryTest.LoadLibraryPreferOwnSymbols fails under trunk tsan/msan

Project Member Reported by thakis@chromium.org, Mar 26 2017

Issue description

https://codereview.chromium.org/2771563002 -- tsan try bot consistently fails with new clang. Error:

==24014==You are trying to dlopen a /b/s/w/irgv6LsP/out/Release/libtest_shared_library.so shared library with RTLD_DEEPBIND flag which is incompatibe with sanitizer runtime (see https://github.com/google/sanitizers/issues/611 for details). If you want to run /b/s/w/irgv6LsP/out/Release/libtest_shared_library.so library under sanitizers please remove RTLD_DEEPBIND from dlopen flags.


jyknight, kcc: This test used to work fine (or looked like it worked fine) for years. You can see the test here: https://cs.chromium.org/chromium/src/base/native_library_unittest.cc?q=LoadLibraryPreferOwnSymbols+package:%5Echromium$&dr=CSs&l=108

What's the right way forward here? Should we disable this test under tsan? Is the new check in tsan too aggressive?
 

Comment 1 by thakis@chromium.org, Mar 27 2017

Left a comment on https://reviews.llvm.org/D30504 and will disable the test for now.
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3b3794c961885d1c714213f751846e4539f04042

commit 3b3794c961885d1c714213f751846e4539f04042
Author: thakis <thakis@chromium.org>
Date: Mon Mar 27 17:00:01 2017

Roll clang 296321:298539.

Ran `tools/clang/scripts/upload_revision.py 298539`.

The roll makes the tsan runtime abort on RTLD_DEEPBIND,
so disable one test under tsan that used that.

BUG= 703833 , 705255 
TBR=hans

Review-Url: https://codereview.chromium.org/2771563002
Cr-Commit-Position: refs/heads/master@{#459802}

[modify] https://crrev.com/3b3794c961885d1c714213f751846e4539f04042/base/native_library_unittest.cc
[modify] https://crrev.com/3b3794c961885d1c714213f751846e4539f04042/tools/clang/scripts/update.py

Comment 3 by a...@chromium.org, Mar 30 2017

Status: Untriaged (was: Unconfirmed)
Summary: Regression: NativeLibraryTest.LoadLibraryPreferOwnSymbols fails under trunk tsan/msan (was: Regression: NativeLibraryTest.LoadLibraryPreferOwnSymbols fails under trunk tsan)
Linux MSan Tests is burning with this, too.
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 31 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b0c09591d4f59f728515a628f25e78b19b78c09c

commit b0c09591d4f59f728515a628f25e78b19b78c09c
Author: avi <avi@chromium.org>
Date: Fri Mar 31 01:24:12 2017

The msan runtime aborts on RTLD_DEEPBIND, so disable the test under msan that does that.

BUG= 705255 
TBR=thakis@chromium.org

Review-Url: https://codereview.chromium.org/2787053003
Cr-Commit-Position: refs/heads/master@{#460979}

[modify] https://crrev.com/b0c09591d4f59f728515a628f25e78b19b78c09c/base/native_library_unittest.cc

Status: Archived (was: Untriaged)
Archiving P3s older than 1 year with no owner or component.

Sign in to add a comment