[MERGE chakra-core#2206 @MikeHolman] fix issue where LocalAlloc call was passing page count instead of byte count

MikeHolman · MikeHolman · commit 5d9aac424e8c · 2016-12-14T18:16:56.000-08:00
Merge pull request chakra-core#2206 from MikeHolman:localallocfix While writing this API I changed parameter from being page count to byte size, but missed updating this call site
diff --git a/lib/Common/Memory/PageAllocator.cpp b/lib/Common/Memory/PageAllocator.cpp
@@ -1592,17 +1592,11 @@ PageAllocatorBase<TVirtualAlloc, TSegment, TPageSegment>::MemSetLocal(_In_ void
 }
 
 #if _WIN32
-// SegmentBase<SectionAllocWrapper>, PageSegmentBase<SectionAllocWrapper>
 template<>
 void
 PageAllocatorBase<SectionAllocWrapper>::MemSetLocal(_In_ void *dst, int val, size_t sizeInBytes)
 {
-    size_t pages = sizeInBytes / AutoSystemInfo::PageSize;
-    if (sizeInBytes % AutoSystemInfo::PageSize != 0)
-    {
-        ++pages;
-    }
-    LPVOID localAddr = this->GetVirtualAllocator()->AllocLocal(dst, pages);
+    LPVOID localAddr = this->GetVirtualAllocator()->AllocLocal(dst, sizeInBytes);
     if (localAddr == nullptr)
     {
         MemoryOperationLastError::CheckProcessAndThrowFatalError(this->processHandle);
@@ -1618,12 +1612,7 @@ template<>
 void
 PageAllocatorBase<PreReservedSectionAllocWrapper>::MemSetLocal(_In_ void *dst, int val, size_t sizeInBytes)
 {
-    size_t pages = sizeInBytes / AutoSystemInfo::PageSize;
-    if (sizeInBytes % AutoSystemInfo::PageSize != 0)
-    {
-        ++pages;
-    }
-    LPVOID localAddr = this->GetVirtualAllocator()->AllocLocal(dst, pages);
+    LPVOID localAddr = this->GetVirtualAllocator()->AllocLocal(dst, sizeInBytes);
     if (localAddr == nullptr)
     {
         MemoryOperationLastError::CheckProcessAndThrowFatalError(this->processHandle);
