commit 035f739aa4c20fb6c1aeea78e2adaa3b7547f79b
author Tom Sepez <tsepez@chromium.org> Thu Jun 17 21:47:03 2021 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jun 17 21:47:03 2021 +0000
tree b13e1c0f939bcd7a71fa0ebeb7f9ef0bcef02a47
parent 7832563faa4ba94c69ed7da3beb8e0da352d9fbd

Add test for maximum WeightTable size.

Change-Id: I48fa7f60dc579f674ed1d35d2c0e1faa8bad0a59
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/82172
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>

core/fxge/dib/cstretchengine_unittest.cpp

diff --git a/core/fxge/dib/cstretchengine_unittest.cpp b/core/fxge/dib/cstretchengine_unittest.cpp
index 2053155..888a3d9 100644
--- a/core/fxge/dib/cstretchengine_unittest.cpp
+++ b/core/fxge/dib/cstretchengine_unittest.cpp
@@ -16,6 +16,10 @@
 
 namespace {
 
+// Discovered experimentally
+constexpr uint32_t kTooBigSrcLen = 20;
+constexpr uint32_t kTooBigDestLen = 32 * 1024 * 1024 + 1;
+
 uint32_t PixelWeightSum(const CStretchEngine::PixelWeight* weights) {
   uint32_t sum = 0;
   for (int i = weights->m_SrcStart; i <= weights->m_SrcEnd; ++i) {
@@ -141,3 +145,11 @@
   CStretchEngine::WeightTable table;
   ASSERT_TRUE(table.CalculateWeights(0, 0, 0, 100, 0, 100, options));
 }
+
+TEST(CStretchEngine, TooManyWeights) {
+  FXDIB_ResampleOptions options;
+  CStretchEngine::WeightTable table;
+  ASSERT_FALSE(table.CalculateWeights(kTooBigDestLen, 0, kTooBigDestLen,
+                                      kTooBigSrcLen, 0, kTooBigSrcLen,
+                                      options));
+}