| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the thread receives a signal concurrently with PTRACE_ATTACH,
we can get notification about the signal before notification about stop.
In such case we need to forward the signal to the thread, otherwise
the signal will be missed (as we do PTRACE_DETACH with arg=0) and
any logic relying on signals will break. After forwarding we need to
continue to wait for stopping, because the thread is not stopped yet.
We do ignore delivery of SIGSTOP, because we want to make stop-the-world
as invisible as possible.
http://reviews.llvm.org/D7723
--This line, and those below, will be ignored--
M lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cc
M test/tsan/signal_segv_handler.cc
llvm-svn: 229832
|
|
|
|
|
|
|
|
|
|
|
|
| |
Patch by Sagar Thakur
Reviewers: petarj, earthdok, kcc.
Subscribers: samsonov, dsanders, mohit.bhakkad, Anand.Takale, llvm-commits.
Differential Revision: http://reviews.llvm.org/D7013
llvm-svn: 229830
|
|
|
|
|
|
|
|
|
|
| |
Long story short: stop-the-world briefly resets SIGSEGV handler to SIG_DFL.
This breaks programs that handle and continue after SIGSEGV (namely JVM).
See the test and comments for details.
http://reviews.llvm.org/D7722
llvm-svn: 229678
|
|
|
|
|
|
|
| |
As per discussion with David Blaikie here:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20150216/260289.html
llvm-svn: 229657
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enabling internal ptrace for mips, which fixes some
ptrace related tests. Along with this fixing some
other failures.
Reviewers: Reviewers: eugenis, kcc, samsonov
Subscribers: dsanders, sagar, lldb-commits
Differential Revision: http://reviews.llvm.org/D7332
llvm-svn: 229656
|
|
|
|
|
|
|
|
|
| |
This reverts commit r229556.
Reverting this for now as internal apple builds rely on this
functionality.
llvm-svn: 229585
|
|
|
|
|
|
|
|
|
| |
In general, this is a reasonable warning, except real_pthread_create is
weak and can be null. The existing usage is correct as it the function
is declared with SANITIZER_WEAK, but MSVC can't know that because it is
defined to nothing on Windows.
llvm-svn: 229562
|
|
|
|
|
|
|
|
| |
They autotools build has a number of missing features, supports less
OS, architectures, build configurations, doesn't have any tests and
is hard to support in sync with CMake build.
llvm-svn: 229556
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
always available, and nothing in sanitizer_platform_limits_posix.cc uses
any of its definitions.
See also: https://svnweb.freebsd.org/changeset/base/277201
Reviewers: kcc, kutuzov.viktor.84, samsonov
Reviewed By: samsonov
Differential Revision: http://reviews.llvm.org/D7486
llvm-svn: 229518
|
|
|
|
|
|
| |
GetTls is not defined in Go mode.
llvm-svn: 229395
|
|
|
|
|
|
|
|
| |
mingw gcc complains:
warning: 'err' may be used uninitialized in this function [-Wmaybe-uninitialized]
Printf("Failed to read options from '%s': error %d\n", value, err);
llvm-svn: 229392
|
|
|
|
|
|
|
| |
The function uses EnumProcessModules, which requires psapi.lib,
which is not linked with tsan on windows.
llvm-svn: 229390
|
|
|
|
| |
llvm-svn: 228893
|
|
|
|
|
|
|
|
|
| |
and even before that, it was never implemented. Just define it to zero
instead, so compiler-rt can compile on FreeBSD 11 and later.
Differential Revision: http://reviews.llvm.org/D7485
llvm-svn: 228871
|
|
|
|
| |
llvm-svn: 228702
|
|
|
|
|
|
| |
This should fix symbolization in those cases when the .exe file is moved together with the .pdb
llvm-svn: 228701
|
|
|
|
|
|
|
|
|
|
|
|
| |
using non-absolute paths
MaybeReexec() in asan_mac.cc checks for presence of the ASan dylib in DYLD_INSERT_LIBRARIES, and if it is there, it will process this env. var. and remove the dylib from its value, so that spawned children don't have this variable set. However, the current implementation only works when using a canonical absolute path to the dylib, it fails to remove the dylib for example when using @executable_path.
This patch changes the processing of DYLD_INSERT_LIBRARIES to comparing values only based on filenames (ignoring directories).
Reviewed at http://reviews.llvm.org/D7160
llvm-svn: 228392
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D7342
llvm-svn: 228125
|
|
|
|
|
|
| |
coverage instrumentation as calls.
llvm-svn: 228102
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D7294
llvm-svn: 227959
|
|
|
|
| |
llvm-svn: 227851
|
|
|
|
| |
llvm-svn: 227803
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227795
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227794
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227793
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227792
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227791
|
|
|
|
|
|
|
|
|
| |
arrays
Original commit: http://reviews.llvm.org/rL226440
Related review: http://reviews.llvm.org/D6892
llvm-svn: 227789
|
|
|
|
|
|
| |
This reverts commit r227633. SetEnv was prematurely deleted.
llvm-svn: 227665
|
|
|
|
| |
llvm-svn: 227634
|
|
|
|
| |
llvm-svn: 227633
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 227572
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D7233
llvm-svn: 227570
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D7171
llvm-svn: 227560
|
|
|
|
|
|
| |
move GetRSS to sanitizer_linux_libcdep.cc (no change in the function itself)
llvm-svn: 227416
|
|
|
|
|
|
| |
while it is running (single-threaded). Also expose the current coverage set to the process.
llvm-svn: 227387
|
|
|
|
|
|
| |
It was fixed the right way in r227195.
llvm-svn: 227196
|
|
|
|
|
|
|
|
|
| |
See https://code.google.com/p/address-sanitizer/issues/detail?id=361
It's still not clear whether the values are correct in all cases, but at least
this should unbreak our bots.
llvm-svn: 226938
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The interceptor of ioctl is using a non-standard prototype:
INTERCEPTOR(int, ioctl, int d, unsigned request, void *arg)
At least on OS X, the request argument should be unsigned long and not
just unsigned, and also instead of the last argument (arg), the function
should be accepting a variable number of arguments, so the prototype
should be:
int ioctl(int fildes, unsigned long request, ...);
We can still keep using `unsigned` internally to save space, because we
know that all possible values of `request` will fit into it.
Reviewed at http://reviews.llvm.org/D7038
llvm-svn: 226926
|
|
|
|
| |
llvm-svn: 226913
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 226910
|
|
|
|
|
|
|
|
|
|
| |
This patch is a proposed solution for https://code.google.com/p/address-sanitizer/issues/detail?id=375:
When the stacktraces are captured and printed by ASan itself, they are fine, but when the program has already printed the report (or is just printing it), capturing a stacktrace via other means is broken. "Other means" include OS X CrashReporter, debuggers or calling backtrace() within the program. For example calling backtrace() from a sanitizer_set_death_callback function prints a very truncated stacktrace.
Reviewed at http://reviews.llvm.org/D7103
llvm-svn: 226878
|
|
|
|
| |
llvm-svn: 226827
|
|
|
|
| |
llvm-svn: 226825
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By attaching an extra integer tag to heap origins, we are able
to distinguish between uninits
- created by heap allocation,
- created by heap deallocation (i.e. use-after-free),
- created by __msan_allocated_memory call,
- etc.
See https://code.google.com/p/memory-sanitizer/issues/detail?id=35.
llvm-svn: 226821
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D7084
llvm-svn: 226796
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 226794
|
|
|
|
|
|
| |
Committed unreviewed with permission.
llvm-svn: 226649
|
|
|
|
|
|
| |
Differential Revision: http://reviews.llvm.org/D6968
llvm-svn: 226648
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
aarch64-linux kernel has configurable 39, 42 or 47 bit virtual address
space. Most distros AFAIK use 42-bit VA right now, but there are also
39-bit VA users too. The ppc64 handling can be used for this just fine
and support all the 3 sizes.
There are other issues, like allocator32 not really being able to support
the larger addres spaces, and hardcoded 39-bit address space size in other
macros.
Patch by Jakub Jelinek.
llvm-svn: 226639
|