[PATCH] D38593: [sanitizer] Update scudo to use new API

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 13:27:00 PST 2017


cryptoad added inline comments.


================
Comment at: lib/scudo/scudo_allocator_secondary.h:105
     }
-    UnmapOrDie(reinterpret_cast<void *>(Header->MapBeg), Header->MapSize);
+    UnmapOrDie(reinterpret_cast<void *>(StoredRange->base()),
+               StoredRange->size());
----------------
Didn't catch that one before: but shouldn't that be StoredRange->Unmap()?


https://reviews.llvm.org/D38593





More information about the llvm-commits mailing list