diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-01-27 12:42:36 -0800 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-01-27 12:42:36 -0800 |
| commit | 26b8a5b56c255618097cb3a26f640ce6cafce56f (patch) | |
| tree | c9ddbb1ad45d9542a0621182454ca3e149c32cba | |
| parent | 20af0c082a94a006d1b6dbbf2d133dca6e43d05f (diff) | |
| download | pdfium-26b8a5b56c255618097cb3a26f640ce6cafce56f.tar.xz | |
Merge to XFA: Add embedder test for stream length beyond end of file.
Original Review URL: https://codereview.chromium.org/878523003
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/876393003
| -rw-r--r-- | BUILD.gn | 1 | ||||
| -rw-r--r-- | fpdfsdk/src/fpdftext_embeddertest.cpp | 23 | ||||
| -rw-r--r-- | pdfium.gyp | 1 | ||||
| -rw-r--r-- | testing/resources/bug_57.in | 48 | ||||
| -rw-r--r-- | testing/resources/bug_57.pdf | 57 |
5 files changed, 130 insertions, 0 deletions
@@ -1565,6 +1565,7 @@ test("pdfium_embeddertests") { sources = [ "fpdfsdk/src/fpdf_dataavail_embeddertest.cpp", "fpdfsdk/src/fpdfdoc_embeddertest.cpp", + "fpdfsdk/src/fpdftext_embeddertest.cpp", "fpdfsdk/src/fpdfview_embeddertest.cpp", "testing/embedder_test.cpp", "testing/embedder_test.h", diff --git a/fpdfsdk/src/fpdftext_embeddertest.cpp b/fpdfsdk/src/fpdftext_embeddertest.cpp new file mode 100644 index 0000000000..3198fe0f86 --- /dev/null +++ b/fpdfsdk/src/fpdftext_embeddertest.cpp @@ -0,0 +1,23 @@ +// Copyright 2015 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. + +#include "../../testing/embedder_test.h" +#include "../../fpdfsdk/include/fpdfview.h" +#include "../../fpdfsdk/include/fpdftext.h" +#include "testing/gtest/include/gtest/gtest.h" + +class FPDFTextEmbeddertest : public EmbedderTest { +}; + +// Test that the page has characters despite a bad stream length. +TEST_F(FPDFTextEmbeddertest, StreamLengthPastEndOfFile) { + EXPECT_TRUE(OpenDocument("testing/resources/bug_57.pdf")); + FPDF_FORMHANDLE form_handle = SetFormFillEnvironment(); + FPDF_PAGE page = LoadPage(0, form_handle); + EXPECT_NE(nullptr, page); + FPDF_TEXTPAGE textpage = FPDFText_LoadPage(page); + EXPECT_NE(nullptr, textpage); + EXPECT_EQ(13, FPDFText_CountChars(textpage)); + ClearFormFillEnvironment(form_handle); +} diff --git a/pdfium.gyp b/pdfium.gyp index 8dba6afc0e..425327ed0f 100644 --- a/pdfium.gyp +++ b/pdfium.gyp @@ -922,6 +922,7 @@ 'sources': [ 'fpdfsdk/src/fpdf_dataavail_embeddertest.cpp', 'fpdfsdk/src/fpdfdoc_embeddertest.cpp', + 'fpdfsdk/src/fpdftext_embeddertest.cpp', 'fpdfsdk/src/fpdfview_embeddertest.cpp', 'testing/embedder_test.cpp', 'testing/embedder_test.h', diff --git a/testing/resources/bug_57.in b/testing/resources/bug_57.in new file mode 100644 index 0000000000..0825efdfa8 --- /dev/null +++ b/testing/resources/bug_57.in @@ -0,0 +1,48 @@ +{{header}} +{{object 1 0}} << + /Type /Catalog + /Pages 2 0 R +>> +{{object 2 0}} << + /Type /Pages + /MediaBox [ 0 0 200 200 ] + /Count 1 + /Kids [ 3 0 R ] +>> +endobj +{{object 3 0}} << + /Type /Page + /Parent 2 0 R + /Resources << + /Font << + /F1 4 0 R + >> + >> + /Contents 5 0 R +>> +endobj +{{object 4 0}} << + /Type /Font + /Subtype /Type1 + /BaseFont /Times-Roman +>> +endobj +% page content with streamlength past end-of-file. +{{object 5 0}} << + /Length 9999 +>> +stream +BT +70 50 TD +/F1 12 Tf +(Hello, world!) Tj +ET +endstream +endobj +{{xref}} +trailer << + /Size 6 + /Root 1 0 R +>> +{{startxref}} +%%EOF diff --git a/testing/resources/bug_57.pdf b/testing/resources/bug_57.pdf new file mode 100644 index 0000000000..d954c43f54 --- /dev/null +++ b/testing/resources/bug_57.pdf @@ -0,0 +1,57 @@ +%PDF-1.7 +% ò¤ô +1 0 obj << + /Type /Catalog + /Pages 2 0 R +>> +2 0 obj << + /Type /Pages + /MediaBox [ 0 0 200 200 ] + /Count 1 + /Kids [ 3 0 R ] +>> +endobj +3 0 obj << + /Type /Page + /Parent 2 0 R + /Resources << + /Font << + /F1 4 0 R + >> + >> + /Contents 5 0 R +>> +endobj +4 0 obj << + /Type /Font + /Subtype /Type1 + /BaseFont /Times-Roman +>> +endobj +% page content with streamlength past end-of-file. +5 0 obj << + /Length 9999 +>> +stream +BT +70 50 TD +/F1 12 Tf +(Hello, world!) Tj +ET +endstream +endobj +xref +0 6 +0000000000 65536 f +0000000015 00000 n +0000000061 00000 n +0000000154 00000 n +0000000280 00000 n +0000000409 00000 n +trailer << + /Size 6 + /Root 1 0 R +>> +startxref +506 +%%EOF |