[PATCH] D30504: [sanitizer] Bail out with warning if user dlopens shared library with RTLD_DEEPBIND flag

Maxim Ostapenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 28 04:18:29 PDT 2017


m.ostapenko added a comment.

In https://reviews.llvm.org/D30504#711277, @thakis wrote:

> We have a test in chromium that loads a small so file and calls some of the functions therein. There are no allocations in the so file. The test used to run fine under tsan, now it fails with the error message you added. I'll disable the test under tsan, but maybe others out there where also calling non-allocating so's without problems before this change.


Oh, sorry. Perhaps we can remove Die()? The check still should fire a warning IMHO.


Repository:
  rL LLVM

https://reviews.llvm.org/D30504





More information about the llvm-commits mailing list