diff options
Diffstat (limited to 'WebCore/html/canvas/ArrayBuffer.cpp')
| -rw-r--r-- | WebCore/html/canvas/ArrayBuffer.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/WebCore/html/canvas/ArrayBuffer.cpp b/WebCore/html/canvas/ArrayBuffer.cpp index 2136f64..ee8f149 100644 --- a/WebCore/html/canvas/ArrayBuffer.cpp +++ b/WebCore/html/canvas/ArrayBuffer.cpp @@ -85,10 +85,7 @@ ArrayBuffer::~ArrayBuffer() void* ArrayBuffer::tryAllocate(unsigned numElements, unsigned elementByteSize) { void* result; - // Do not allow 32-bit overflow of the total size. - // FIXME: Why not? The tryFastCalloc function already checks its arguments, - // and will fail if there is any overflow, so why should we include a - // redudant unnecessarily restrictive check here? + // Do not allow 32-bit overflow of the total size if (numElements) { unsigned totalSize = numElements * elementByteSize; if (totalSize / numElements != elementByteSize) |