Remove a useless assignment in CPDF_Parser::ParseCrossRefV4(). Fix the variable name too. Change-Id: I8e3fe9ba03dd9a14dca5089f5450220fc0b8ff56 Reviewed-on: https://pdfium-review.googlesource.com/c/47552 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/core/fpdfapi/parser/cpdf_parser.cpp b/core/fpdfapi/parser/cpdf_parser.cpp index dd885bc..5ddae2d 100644 --- a/core/fpdfapi/parser/cpdf_parser.cpp +++ b/core/fpdfapi/parser/cpdf_parser.cpp
@@ -507,15 +507,14 @@ return false; std::vector<CrossRefObjData> result_objects; while (1) { - FX_FILESIZE SavedPos = m_pSyntax->GetPos(); + FX_FILESIZE saved_pos = m_pSyntax->GetPos(); bool bIsNumber; ByteString word = m_pSyntax->GetNextWord(&bIsNumber); - if (word.IsEmpty()) { + if (word.IsEmpty()) return false; - } if (!bIsNumber) { - m_pSyntax->SetPos(SavedPos); + m_pSyntax->SetPos(saved_pos); break; } @@ -525,7 +524,6 @@ uint32_t count = m_pSyntax->GetDirectNum(); m_pSyntax->ToNextWord(); - SavedPos = m_pSyntax->GetPos(); if (!ParseAndAppendCrossRefSubsectionData( start_objnum, count, out_objects ? &result_objects : nullptr)) {