[PATCH] D54190: [sanitizer] Only set soft coredump limit.

[Evgenii Stepanov via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL346284: [sanitizer] Only set soft coredump limit. (authored by eugenis, committed by ).
Herald added a subscriber: delcypher.

Changed prior to commit:
  https://reviews.llvm.org/D54190?vs=172880&id=172881#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D54190

Files:
  compiler-rt/trunk/lib/sanitizer_common/sanitizer_posix_libcdep.cc
  compiler-rt/trunk/test/sanitizer_common/TestCases/corelimit.cc


Index: compiler-rt/trunk/lib/sanitizer_common/sanitizer_posix_libcdep.cc
===================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_posix_libcdep.cc
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_posix_libcdep.cc
@@ -94,10 +94,12 @@
 }
 
 static void setlim(int res, rlim_t lim) {
-  // The following magic is to prevent clang from replacing it with memset.
-  volatile struct rlimit rlim;
+  struct rlimit rlim;
+  if (getrlimit(res, const_cast<struct rlimit *>(&rlim))) {
+    Report("ERROR: %s getrlimit() failed %d\n", SanitizerToolName, errno);
+    Die();
+  }
   rlim.rlim_cur = lim;
-  rlim.rlim_max = lim;
   if (setrlimit(res, const_cast<struct rlimit *>(&rlim))) {
     Report("ERROR: %s setrlimit() failed %d\n", SanitizerToolName, errno);
     Die();
Index: compiler-rt/trunk/test/sanitizer_common/TestCases/corelimit.cc
===================================================================
--- compiler-rt/trunk/test/sanitizer_common/TestCases/corelimit.cc
+++ compiler-rt/trunk/test/sanitizer_common/TestCases/corelimit.cc
@@ -10,7 +10,7 @@
   getrlimit(RLIMIT_CORE, &lim_core);
   void *p;
   if (sizeof(p) == 8) {
-    assert(0 == lim_core.rlim_max);
+    assert(0 == lim_core.rlim_cur);
   }
   return 0;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54190.172881.patch
Type: text/x-patch
Size: 1292 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181107/4b5f5d69/attachment.bin>