diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-01-23 15:05:43 -0800 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-01-23 15:05:43 -0800 |
commit | e0bbe4aac90e56950f28e322c3d6192c35af5ea0 (patch) | |
tree | 55cc9af3afa38333b13a55810d85cd788e6196f8 /testing/embedder_test.h | |
parent | af9be4fe77e1067f1cc265de8aae547f6358bc44 (diff) | |
download | pdfium-e0bbe4aac90e56950f28e322c3d6192c35af5ea0.tar.xz |
Fix null crash in CheckTrailer.
We are making checks in the incorrect order. Also adds two test
cases, one for the this crash, and another for the original issue
that motivated the patch.
Original Patch by Bo at https://codereview.chromium.org/866003003/
BUG=450871
R=bo_xu@foxitsoftware.com
Review URL: https://codereview.chromium.org/872563002
Diffstat (limited to 'testing/embedder_test.h')
-rw-r--r-- | testing/embedder_test.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/testing/embedder_test.h b/testing/embedder_test.h index 48ea415e19..3eb3be606f 100644 --- a/testing/embedder_test.h +++ b/testing/embedder_test.h @@ -65,7 +65,7 @@ class EmbedderTest : public ::testing::Test { // is prohibited after this call is made. virtual void UnloadPage(FPDF_PAGE page, FPDF_FORMHANDLE form); - private: + protected: FPDF_DOCUMENT document_; FPDF_AVAIL avail_; FX_DOWNLOADHINTS hints_; |