diff options
author | Evgeniy Stepanov <eugeni.stepanov@gmail.com> | 2014-06-24 11:50:26 +0000 |
---|---|---|
committer | Evgeniy Stepanov <eugeni.stepanov@gmail.com> | 2014-06-24 11:50:26 +0000 |
commit | 3d551f27a4d33a82ac46070977c67d6f78067f28 (patch) | |
tree | 45ecbfaeaa1e7d7306e2adc5ecf36ab29fe3601f /lib/msan/msan_interceptors.cc | |
parent | 26729fe0a77f1f7137bf88fef94ecc52ab56961a (diff) |
[msan] Fix origin copying.
Conditions for the first and the last origin value in range were wrong.
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@211585 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/msan/msan_interceptors.cc')
-rw-r--r-- | lib/msan/msan_interceptors.cc | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/lib/msan/msan_interceptors.cc b/lib/msan/msan_interceptors.cc index 854c20534..c7f412484 100644 --- a/lib/msan/msan_interceptors.cc +++ b/lib/msan/msan_interceptors.cc @@ -1389,7 +1389,7 @@ void CopyOrigin(void *dst, const void *src, uptr size, StackTrace *stack) { uptr beg = d & ~3UL; // Copy left unaligned origin if that memory is poisoned. if (beg < d) { - u32 o = GetOriginIfPoisoned(beg, d - beg); + u32 o = GetOriginIfPoisoned((uptr)src, d - beg); if (o) { if (__msan_get_track_origins() > 1) o = ChainOrigin(o, stack); *(u32 *)MEM_TO_ORIGIN(beg) = o; @@ -1397,15 +1397,14 @@ void CopyOrigin(void *dst, const void *src, uptr size, StackTrace *stack) { beg += 4; } - uptr end = (d + size + 3) & ~3UL; + uptr end = (d + size) & ~3UL; // Copy right unaligned origin if that memory is poisoned. - if (end > d + size) { - u32 o = GetOriginIfPoisoned(d + size, end - d - size); + if (end < d + size) { + u32 o = GetOriginIfPoisoned((uptr)src + (end - d), (d + size) - end); if (o) { if (__msan_get_track_origins() > 1) o = ChainOrigin(o, stack); - *(u32 *)MEM_TO_ORIGIN(end - 4) = o; + *(u32 *)MEM_TO_ORIGIN(end) = o; } - end -= 4; } if (beg < end) { |