diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-06-12 19:26:15 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-12 19:26:15 +0000 |
| commit | 8128485d0169a8dcbe14289a868ff3c27440bfcf (patch) | |
| tree | fd80bed666b8861efe40396bb599c948eb8c3c91 /fpdfsdk/cpdfsdk_helpers.cpp | |
| parent | a3843c99e7a631b2fbf87bbc12834e61bfb7b6f2 (diff) | |
| download | pdfium-8128485d0169a8dcbe14289a868ff3c27440bfcf.tar.xz | |
CheckUnSupportError() always called with FPDF_ERR_SUCCESS.
So remove the argument, and the logic dependent upon it being otherwise.
Rename function to more accurately reflect its purpose. Remove needless
null check while we're at it.
Change-Id: I7e119429fd159334eabcc250b5dac215580882ea
Reviewed-on: https://pdfium-review.googlesource.com/34950
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_helpers.cpp')
| -rw-r--r-- | fpdfsdk/cpdfsdk_helpers.cpp | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/fpdfsdk/cpdfsdk_helpers.cpp b/fpdfsdk/cpdfsdk_helpers.cpp index 391e79758f..943e3a3153 100644 --- a/fpdfsdk/cpdfsdk_helpers.cpp +++ b/fpdfsdk/cpdfsdk_helpers.cpp @@ -199,15 +199,7 @@ void CheckUnSupportAnnot(CPDF_Document* pDoc, const CPDF_Annot* pPDFAnnot) { } } -void CheckUnSupportError(CPDF_Document* pDoc, uint32_t err_code) { - // Security - if (err_code == FPDF_ERR_SECURITY) { - RaiseUnSupportError(FPDF_UNSP_DOC_SECURITY); - return; - } - if (!pDoc) - return; - +void ReportUnsupportedFeatures(CPDF_Document* pDoc) { const CPDF_Dictionary* pRootDict = pDoc->GetRoot(); if (pRootDict) { // Portfolios and Packages @@ -236,6 +228,7 @@ void CheckUnSupportError(CPDF_Document* pDoc, uint32_t err_code) { } } } + // SharedForm const CPDF_Stream* pStream = pRootDict->GetStreamFor("Metadata"); if (pStream) { |