| commit | 01e6b8620f867693a4d4a4df3ca8220321c30bc9 | [log] [tgz] |
|---|---|---|
| author | Tom Sepez <tsepez@chromium.org> | Fri Jul 14 01:06:21 2023 +0000 |
| committer | Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> | Fri Jul 14 01:06:21 2023 +0000 |
| tree | ac36a39e80e3ae99dbd5968e52b12ef1bf1b3db7 | |
| parent | 8ea56a4c57fe27db95e062966f460a5d4c68777e [diff] |
Convert bounds-check from DCHECK() to CHECK() in JBig2_ArithDecoder Bounds-checking shouldn't be debug only. -- Remove somewhat redundant DCHECK(). Change-Id: I00cc188dddd4e1e90df1c60914deed377d1ab2be Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/109590 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
diff --git a/core/fxcodec/jbig2/JBig2_ArithDecoder.cpp b/core/fxcodec/jbig2/JBig2_ArithDecoder.cpp index 2555ee2..625ebf2 100644 --- a/core/fxcodec/jbig2/JBig2_ArithDecoder.cpp +++ b/core/fxcodec/jbig2/JBig2_ArithDecoder.cpp
@@ -66,8 +66,7 @@ CJBig2_ArithDecoder::~CJBig2_ArithDecoder() = default; int CJBig2_ArithDecoder::Decode(JBig2ArithCtx* pCX) { - DCHECK(pCX); - DCHECK_LT(pCX->I(), std::size(kQeTable)); + CHECK_LT(pCX->I(), std::size(kQeTable)); const JBig2ArithCtx::JBig2ArithQe& qe = kQeTable[pCX->I()]; m_A -= qe.Qe;