[PATCH] D35694: [scudo] Quarantine overhaul

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 22 12:45:57 PDT 2017


cryptoad updated this revision to Diff 107797.
cryptoad marked 3 inline comments as done.
cryptoad added a comment.

Addressing Aleksey's comments.

If QuarantineSizeMb is set, check that QuarantineChunksUpToSize isn't.
Otherwise proceed with the new flags. Modified the tests accordingly
I left the upper limit checks out of the else clause on purpose, let me know if
that looks like what you were thinking.


https://reviews.llvm.org/D35694

Files:
  lib/scudo/scudo_allocator.cpp
  lib/scudo/scudo_flags.cpp
  lib/scudo/scudo_flags.inc
  test/scudo/overflow.cpp
  test/scudo/quarantine.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35694.107797.patch
Type: text/x-patch
Size: 18016 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170722/7f796c2e/attachment.bin>


More information about the llvm-commits mailing list