Bug 1158445 - posix_madvise() should be used instead of madvise in Po…

…ison code; r=froydnj
mozilla-spidermonkey · Jun 7, 2017 · e55c9d9 · e55c9d9
1 parent ea4dd2f
commit e55c9d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/mfbt/Poison.cpp b/mfbt/Poison.cpp
@@ -129,7 +129,7 @@ ReleaseRegion(void* aRegion, uintptr_t aSize)
 static bool
 ProbeRegion(uintptr_t aRegion, uintptr_t aSize)
 {
-  if (madvise(reinterpret_cast<void*>(aRegion), aSize, MADV_NORMAL)) {
+  if (posix_madvise(reinterpret_cast<void*>(aRegion), aSize, POSIX_MADV_NORMAL)) {
     return true;
   } else {
     return false;

diff --git a/mfbt/tests/TestPoisonArea.cpp b/mfbt/tests/TestPoisonArea.cpp
@@ -269,7 +269,7 @@ ReleaseRegion(void* aPage)
 static bool
 ProbeRegion(uintptr_t aPage)
 {
-  return !!madvise(reinterpret_cast<void*>(aPage), PageSize(), MADV_NORMAL);
+  return !!posix_madvise(reinterpret_cast<void*>(aPage), PageSize(), POSIX_MADV_NORMAL);
 }
 
 static int