Attachment 72420 Details for Bug 58966 – fix bug in linearization code when first object is a stream

[patch] fix bug in linearization code when first object is a stream

0001-Linearization-first-object-can-be-a-stream.patch (text/plain), 1.08 KB, created by Adrian Johnson on 2013-01-03 05:19:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Adrian Johnson

Created: 2013-01-03 05:19:29 UTC

Size: 1.08 KB

patch

obsolete

>From abb22aaf7eee8d49b568e40b81864b0a0687436a Mon Sep 17 00:00:00 2001
>From: Adrian Johnson <ajohnson@redneon.com>
>Date: Thu, 3 Jan 2013 15:27:36 +1030
>Subject: [PATCH] Linearization: first object can be a stream
>
>Since the check for linearization in Linearization::Linearization is
>reading from the PDF file stream, the Parser object should have
>allowStreams enabled. Otherwise opening PDF files that have a stream
>as the first object will result in a "Command token too long" error
>message.
>---
> poppler/Linearization.cc |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/poppler/Linearization.cc b/poppler/Linearization.cc
>index 33331d1..7d0feb3 100644
>--- a/poppler/Linearization.cc
>+++ b/poppler/Linearization.cc
>@@ -26,7 +26,7 @@ Linearization::Linearization (BaseStream *str)
>   str->reset();
>   obj1.initNull();
>   parser = new Parser(NULL,
>-      new Lexer(NULL, str->makeSubStream(str->getStart(), gFalse, 0, &obj1)),
>+      new Lexer(NULL, str->makeSubStream(str->getStart(), gTrue, 0, &obj1)),
>       gFalse);
>   parser->getObj(&obj1);
>   parser->getObj(&obj2);
>-- 
>1.7.10.4
>

Actions: View | Splinter Review

Attachments on bug 58966: 72413 | 72420 | 72423 | 72424

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.