diff options
| author | tsepez <tsepez@chromium.org> | 2016-10-25 15:16:40 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-10-25 15:16:40 -0700 |
| commit | 04c1009af887eb5d3a183dd26421f644f63d8b77 (patch) | |
| tree | e0400b66f317c60447f127dcc17307f3c27c513c /fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp | |
| parent | 530c7acf7aa11ff8db8743af0d62b39cf36bf539 (diff) | |
| download | pdfium-chromium/2901.tar.xz | |
Fix some return FALSE in functions that return pointerschromium/2901
This previously compiled because FALSE is #defined to 0, and 0
is a perfectly fine value for a pointer.
Review-Url: https://codereview.chromium.org/2448943003
Diffstat (limited to 'fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp')
| -rw-r--r-- | fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp index 13b79baf9b..2a2bb3c48d 100644 --- a/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp +++ b/fpdfsdk/fpdfxfa/cpdfxfa_docenvironment.cpp @@ -709,16 +709,16 @@ IFX_SeekableReadStream* CPDFXFA_DocEnvironment::OpenLinkedFile( const CFX_WideString& wsLink) { CPDFSDK_FormFillEnvironment* pFormFillEnv = m_pDocument->GetFormFillEnv(); if (!pFormFillEnv) - return FALSE; + return nullptr; CFX_ByteString bs = wsLink.UTF16LE_Encode(); int len = bs.GetLength(); FPDF_FILEHANDLER* pFileHandler = pFormFillEnv->OpenFile(0, (FPDF_WIDESTRING)bs.GetBuffer(len), "rb"); bs.ReleaseBuffer(len); - if (!pFileHandler) return nullptr; + return new CFPDF_FileStream(pFileHandler); } |