summaryrefslogtreecommitdiff
path: root/fxjs/fxjs_v8.cpp
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2017-03-15 15:24:57 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-03-15 22:50:56 +0000
commit972eb5c6843b235e5414c86c41e962151a94870c (patch)
treeab0678b757c6da4b7d11219192eae50999f65ee9 /fxjs/fxjs_v8.cpp
parentc467d4619ebe0bae9a87b667ca9a06f576138f68 (diff)
downloadpdfium-972eb5c6843b235e5414c86c41e962151a94870c.tar.xz
Add array_buffer JS test.
JS Array Buffers are the first candidate to be allocated from PartitionAlloc when it becomes available, so add test first. Presently, we will return as large an array buffer as the system can handle; this is generally a bad idea so limit them to 256MB and test that we handle failure. Change-Id: I205745a7938d69eb32ac883b90824f2f9e584ec7 Reviewed-on: https://pdfium-review.googlesource.com/3065 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs/fxjs_v8.cpp')
-rw-r--r--fxjs/fxjs_v8.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/fxjs/fxjs_v8.cpp b/fxjs/fxjs_v8.cpp
index b0e1a1b260..5f9426b643 100644
--- a/fxjs/fxjs_v8.cpp
+++ b/fxjs/fxjs_v8.cpp
@@ -144,11 +144,11 @@ static v8::Local<v8::ObjectTemplate> GetGlobalObjectTemplate(
}
void* FXJS_ArrayBufferAllocator::Allocate(size_t length) {
- return calloc(1, length);
+ return length <= kMaxAllowedBytes ? calloc(1, length) : nullptr;
}
void* FXJS_ArrayBufferAllocator::AllocateUninitialized(size_t length) {
- return malloc(length);
+ return length < kMaxAllowedBytes ? malloc(length) : nullptr;
}
void FXJS_ArrayBufferAllocator::Free(void* data, size_t length) {