in the attempt to fix 627393, changed IFX_FileRead's readBlock to return the length it reads

xfa/fgas/crt/fgas_stream.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "xfa/fgas/crt/fgas_stream.h"
 
 #if _FX_OS_ == _FX_WIN32_DESKTOP_ || _FX_OS_ == _FX_WIN32_MOBILE_ || \
     _FX_OS_ == _FX_WIN64_
@@ skipping 278 matching lines @@
   void InitStream();
 };
 
 class CFGAS_FileRead : public IFX_FileRead {
  public:
   CFGAS_FileRead(IFX_Stream* pStream, FX_BOOL bReleaseStream);
   ~CFGAS_FileRead() override;
 
   // IFX_FileRead
   void Release() override;
+  FX_BOOL IsEOF() override;
   FX_FILESIZE GetSize() override;
-  FX_BOOL ReadBlock(void* buffer, FX_FILESIZE offset, size_t size) override;
+  size_t ReadBlock(void* buffer, FX_FILESIZE offset, size_t size) override;
 
  protected:
   FX_BOOL m_bReleaseStream;
   IFX_Stream* m_pStream;
+  FX_FILESIZE m_nCurPos;
 };
 
 int32_t FileLength(FXSYS_FILE* file) {
   ASSERT(file);
 #if _FX_OS_ == _FX_WIN32_DESKTOP_ || _FX_OS_ == _FX_WIN64_
   return _filelength(_fileno(file));
 #else
   int32_t iPos = FXSYS_ftell(file);
   FXSYS_fseek(file, 0, FXSYS_SEEK_END);
   int32_t iLen = FXSYS_ftell(file);
@@ skipping 268 matching lines @@
 }
 FX_BOOL CFX_FileReadStreamImp::IsEOF() const {
   return m_iPosition >= m_iLength;
 }
 int32_t CFX_FileReadStreamImp::ReadData(uint8_t* pBuffer, int32_t iBufferSize) {
   ASSERT(m_pFileRead);
   ASSERT(pBuffer && iBufferSize > 0);
   if (iBufferSize > m_iLength - m_iPosition) {
     iBufferSize = m_iLength - m_iPosition;
   }
-  if (m_pFileRead->ReadBlock(pBuffer, m_iPosition, iBufferSize)) {
+  if (m_pFileRead->ReadBlock(pBuffer, m_iPosition, iBufferSize) ==
+          iBufferSize ||
+      m_pFileRead->IsEOF()) {
     m_iPosition += iBufferSize;
     return iBufferSize;
   }
   return 0;
 }
 int32_t CFX_FileReadStreamImp::ReadString(FX_WCHAR* pStr,
                                           int32_t iMaxLength,
                                           FX_BOOL& bEOS) {
   ASSERT(m_pFileRead);
   ASSERT(pStr && iMaxLength > 0);
@@ skipping 862 matching lines @@
     return IFX_Stream::CreateTextStream(pShared, TRUE);
   }
   return pShared;
 }
 IFX_FileRead* FX_CreateFileRead(IFX_Stream* pBaseStream,
                                 FX_BOOL bReleaseStream) {
   ASSERT(pBaseStream);
   return new CFGAS_FileRead(pBaseStream, bReleaseStream);
 }
 CFGAS_FileRead::CFGAS_FileRead(IFX_Stream* pStream, FX_BOOL bReleaseStream)
-    : m_bReleaseStream(bReleaseStream), m_pStream(pStream) {
+    : m_bReleaseStream(bReleaseStream), m_pStream(pStream), m_nCurPos(0) {
   ASSERT(m_pStream);
 }
 CFGAS_FileRead::~CFGAS_FileRead() {
   if (m_bReleaseStream) {
     m_pStream->Release();
   }
 }
+FX_BOOL CFGAS_FileRead::IsEOF() {
+  return m_nCurPos >= (FX_FILESIZE)m_pStream->GetLength();
+}
 FX_FILESIZE CFGAS_FileRead::GetSize() {
   return (FX_FILESIZE)m_pStream->GetLength();
 }
-FX_BOOL CFGAS_FileRead::ReadBlock(void* buffer,
-                                  FX_FILESIZE offset,
-                                  size_t size) {
+size_t CFGAS_FileRead::ReadBlock(void* buffer,
+                                 FX_FILESIZE offset,
+                                 size_t size) {
   m_pStream->Seek(FX_STREAMSEEK_Begin, (int32_t)offset);
   int32_t iLen = m_pStream->ReadData((uint8_t*)buffer, (int32_t)size);
-  return iLen == (int32_t)size;
+  m_nCurPos = offset + iLen;
+  return iLen;
 }
 
 void CFGAS_FileRead::Release() {
   delete this;
 }