diff options
| author | tsepez <tsepez@chromium.org> | 2016-11-07 15:16:01 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-11-07 15:16:01 -0800 |
| commit | 4e597c8fd5d604273266ea19a1074a2c870ba224 (patch) | |
| tree | 6311f6884e5316cb0c481cc141a08f8984cbfa6e /core/fpdfapi/parser/cpdf_parser.cpp | |
| parent | 0d830c1e0db684d17f9b3c534dec8cecb17b674d (diff) | |
| download | pdfium-4e597c8fd5d604273266ea19a1074a2c870ba224.tar.xz | |
Force compiler to deduce src type for checked_cast<dst, src>.
Otherwise, we might be silently doing an unsafe cast before
applying the check if the actual argument doesn't match the
exact src type.
Review-Url: https://codereview.chromium.org/2484953003
Diffstat (limited to 'core/fpdfapi/parser/cpdf_parser.cpp')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_parser.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/core/fpdfapi/parser/cpdf_parser.cpp b/core/fpdfapi/parser/cpdf_parser.cpp index cf23ec7cd7..ed20cf73f4 100644 --- a/core/fpdfapi/parser/cpdf_parser.cpp +++ b/core/fpdfapi/parser/cpdf_parser.cpp @@ -1015,10 +1015,8 @@ bool CPDF_Parser::LoadCrossRefV5(FX_FILESIZE* pos, bool bMainXRef) { if (startnum < 0) continue; - m_dwXrefStartObjNum = - pdfium::base::checked_cast<uint32_t, int32_t>(startnum); - uint32_t count = - pdfium::base::checked_cast<uint32_t, int32_t>(arrIndex[i].second); + m_dwXrefStartObjNum = pdfium::base::checked_cast<uint32_t>(startnum); + uint32_t count = pdfium::base::checked_cast<uint32_t>(arrIndex[i].second); FX_SAFE_UINT32 dwCaculatedSize = segindex; dwCaculatedSize += count; dwCaculatedSize *= totalWidth; |