commit a1188905b85a84d950f94df0f229b9f1d5784c61
author Lei Zhang <thestig@chromium.org> Thu Oct 11 23:09:10 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Oct 11 23:09:10 2018 +0000
tree 17d39fceb0d26c4313085dd3fad65e5c6f3babe1
parent c7f66edd224446deefe68506b7ebed871eb5c8c2

Use pdfium::Vector2D in CCodec_FaxEncoder ctor.

Instead of manual CHECKs.

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

core/fxcodec/codec/ccodec_faxmodule.cpp

diff --git a/core/fxcodec/codec/ccodec_faxmodule.cpp b/core/fxcodec/codec/ccodec_faxmodule.cpp
index 905f647..6a07d56 100644
--- a/core/fxcodec/codec/ccodec_faxmodule.cpp
+++ b/core/fxcodec/codec/ccodec_faxmodule.cpp
@@ -8,7 +8,6 @@
 
 #include <algorithm>
 #include <iterator>
-#include <limits>
 #include <memory>
 #include <vector>
 
@@ -678,11 +677,9 @@
                                      int height,
                                      int pitch)
     : m_Cols(width), m_Rows(height), m_Pitch(pitch), m_pSrcBuf(src_buf) {
-  CHECK(pitch > 0);
-  CHECK(pitch < std::numeric_limits<int>::max() / 8);
   m_RefLine.resize(pitch);
   std::fill(std::begin(m_RefLine), std::end(m_RefLine), 0xff);
-  m_LineBuf.resize(8 * m_Pitch);
+  m_LineBuf = pdfium::Vector2D<uint8_t>(8, m_Pitch);
   m_DestBuf.SetAllocStep(10240);
 }