Chromium Code Reviews| Index: core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp |
| diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp |
| index 1115a1e560d0ab39bb5104dc6230e2ddb8b13aac..c6e48e74a2cc9ba20a8177a4ef6cf26fb4deac9a 100644 |
| --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp |
| +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp |
| @@ -2975,10 +2975,10 @@ FX_DWORD CPDF_DataAvail::GetObjectSize(FX_DWORD objnum, FX_FILESIZE& offset) { |
| if (!pParser || !pParser->IsValidObjectNumber(objnum)) |
| return 0; |
| - if (pParser->m_V5Type[objnum] == 2) { |
| + if (pParser->m_V5Type[objnum] == 2) |
| objnum = pParser->m_ObjectInfo[objnum].pos; |
|
Lei Zhang
2015/12/25 05:12:21
Changing a passed by value param makes no sense.
|
| - } else if (pParser->m_V5Type[objnum] == 1 || |
| - pParser->m_V5Type[objnum] == 255) { |
| + |
| + if (pParser->m_V5Type[objnum] == 1 || pParser->m_V5Type[objnum] == 255) { |
| offset = pParser->m_ObjectInfo[objnum].pos; |
| if (offset == 0) { |
| return 0; |