Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

This queue is for tickets about the PDF-API2 CPAN distribution.

Report information
The Basics
Id: 120397
Status: resolved
Priority: 0/
Queue: PDF-API2
People
Owner: Nobody in particular
Requestors: 'spro^^*%*^6ut# [...] &$%*c
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 2.032
Attachments
newlines.pdf

History Show all quoted text — Show brief headers
# Sun Feb 26 17:41:13 2017 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Ticket created
Subject: Can’t handle newlines in references
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 643
Download (untitled) / with headers
text/plain 643b
In PDF syntax, an indirect reference consists of three distinct tokens that can be separated by any PDF whitespace, and even comments. For example, this is a syntactically valid indirect reference: 1 %eieio 0 R PDF::API2 does not allow comments at all (based on reading the code; that is not a problem for my PDFs). But it does choke on newlines if the object is long enough that it has not all been read into the file yet. This happens with: 1895 0 obj<</Count 253/Kids[1896 0 R 1 0 R 7 0 R 13 0 R ... etc., with 253 entries. PDF::API2::Basic::PDF::File::readval needs to read more data if it finds what could be a partial reference.
# Sat Jun 24 11:52:14 2017 steve [...] deefs.net - Correspondence added
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 948
Download (untitled) / with headers
text/plain 948b
Thanks for the bug report. I think this should now be working as expected. See t/rt120397.t for the cases that are now being tested -- if anything is missing, add a test and let me know. On Sun Feb 26 17:41:13 2017, SPROUT wrote: Show quoted text
> In PDF syntax, an indirect reference consists of three distinct tokens > that can be separated by any PDF whitespace, and even comments. For > example, this is a syntactically valid indirect reference: > > 1 %eieio > 0 > R > > PDF::API2 does not allow comments at all (based on reading the code; > that is not a problem for my PDFs). But it does choke on newlines if > the object is long enough that it has not all been read into the file > yet. > > This happens with: > > 1895 0 > obj<</Count > 253/Kids[1896 > 0 > R > 1 > 0 > R > 7 > 0 > R > 13 > 0 > R > ... > > etc., with 253 entries. > > PDF::API2::Basic::PDF::File::readval needs to read more data if it > finds what could be a partial reference.
# Sat Jun 24 11:52:15 2017 The RT System itself - Status changed from 'new' to 'open'
# Sat Jun 24 11:52:15 2017 steve [...] deefs.net - Status changed from 'open' to 'patched'
# Sat Jun 24 14:41:07 2017 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
X-RT-Interface: Web
References: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org> <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org>
Content-Type: multipart/mixed; boundary="----------=_1498329667-9033-3"
Message-ID: <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org>
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 0
Content-Disposition: inline
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 819
Download (untitled) / with headers
text/plain 819b
On Sat Jun 24 11:52:14 2017, SSIMMS wrote: Show quoted text
> Thanks for the bug report. > > I think this should now be working as expected. See t/rt120397.t for > the cases that are now being tested -- if anything is missing, add a > test and let me know.
Thank you. I’m afraid it is still not working. Attached is a sample 400-page PDF that it fails on. This PDF may not actually be valid. To keep the file size small, I made the pages Kids array reference the same PDF 400 times. Adobe Reader does not like this file, but there is nothing the specification to suggest that the same page object cannot be referenced multiple times in the Kids array. In any case, it makes a good test. I am not sure where you would put this in the repository, but a simple ok eval { PDF::API2->open("t/newlines.pdf") } will suffice.
MIME-Version: 1.0
Subject: newlines.pdf
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Type: application/pdf; name="newlines.pdf"
Content-Disposition: inline; filename="newlines.pdf"
Content-Transfer-Encoding: base64
Content-Length: 2771
Download newlines.pdf
application/pdf 2.7k

Message body not shown because it is not plain text.

# Sat Jun 24 14:42:10 2017 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org> <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org> <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-12138-1498329730-339.120397-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 1065
Download (untitled) / with headers
text/plain 1k
On Sat Jun 24 14:41:07 2017, SPROUT wrote: Show quoted text
> On Sat Jun 24 11:52:14 2017, SSIMMS wrote:
> > Thanks for the bug report. > > > > I think this should now be working as expected. See t/rt120397.t for > > the cases that are now being tested -- if anything is missing, add a > > test and let me know.
> > Thank you. > > I’m afraid it is still not working. Attached is a sample 400-page PDF > that it fails on. This PDF may not actually be valid. To keep the > file size small, I made the pages Kids array reference the same PDF > 400 times. Adobe Reader does not like this file, but there is nothing > the specification to suggest that the same page object cannot be > referenced multiple times in the Kids array. > > In any case, it makes a good test. I am not sure where you would put > this in the repository, but a simple > > ok eval { PDF::API2->open("t/newlines.pdf") } > > will suffice.
BTW, the error I get is: Can't parse `R 3 0 R 3 0 ... many times over ... 3 0 R' near 1000 length 313. at /Library/Perl/5.12/PDF/API2/Basic/PDF/File.pm line 682.
# Sat Jun 24 15:03:07 2017 steve [...] deefs.net - Correspondence added
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org> <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org> <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-11132-1498330987-883.120397-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 1144
Download (untitled) / with headers
text/plain 1.1k
Can you update to HEAD and try again, please? I made a couple more fixes an hour or so after updating this ticket, and I'm guessing you don't have that commit. The PDF you attached opens fine for me on HEAD (but not on the original fix). On Sat Jun 24 14:41:07 2017, SPROUT wrote: Show quoted text
> On Sat Jun 24 11:52:14 2017, SSIMMS wrote:
> > Thanks for the bug report. > > > > I think this should now be working as expected. See t/rt120397.t for > > the cases that are now being tested -- if anything is missing, add a > > test and let me know.
> > Thank you. > > I’m afraid it is still not working. Attached is a sample 400-page PDF > that it fails on. This PDF may not actually be valid. To keep the > file size small, I made the pages Kids array reference the same PDF > 400 times. Adobe Reader does not like this file, but there is nothing > the specification to suggest that the same page object cannot be > referenced multiple times in the Kids array. > > In any case, it makes a good test. I am not sure where you would put > this in the repository, but a simple > > ok eval { PDF::API2->open("t/newlines.pdf") } > > will suffice.
# Sat Jun 24 17:05:04 2017 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-11132-1498330987-883.120397-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-21575-1488148872-421.0-0-0 [...] rt.cpan.org> <rt-4.0.18-13716-1498319534-1341.120397-0-0 [...] rt.cpan.org> <rt-4.0.18-9033-1498329667-1766.120397-0-0 [...] rt.cpan.org> <rt-4.0.18-11132-1498330987-883.120397-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-3566-1498338304-1612.120397-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 1295
Download (untitled) / with headers
text/plain 1.2k
On Sat Jun 24 15:03:07 2017, SSIMMS wrote: Show quoted text
> Can you update to HEAD and try again, please? I made a couple more > fixes an hour or so after updating this ticket, and I'm guessing you > don't have that commit. > > The PDF you attached opens fine for me on HEAD (but not on the > original fix).
Yes, it works now. Thank you. Show quoted text
> > On Sat Jun 24 14:41:07 2017, SPROUT wrote:
> > On Sat Jun 24 11:52:14 2017, SSIMMS wrote:
> > > Thanks for the bug report. > > > > > > I think this should now be working as expected. See t/rt120397.t > > > for > > > the cases that are now being tested -- if anything is missing, add > > > a > > > test and let me know.
> > > > Thank you. > > > > I’m afraid it is still not working. Attached is a sample 400-page > > PDF > > that it fails on. This PDF may not actually be valid. To keep the > > file size small, I made the pages Kids array reference the same PDF > > 400 times. Adobe Reader does not like this file, but there is > > nothing > > the specification to suggest that the same page object cannot be > > referenced multiple times in the Kids array. > > > > In any case, it makes a good test. I am not sure where you would put > > this in the repository, but a simple > > > > ok eval { PDF::API2->open("t/newlines.pdf") } > > > > will suffice.
# Sun Jul 02 23:47:06 2017 steve [...] deefs.net - Status changed from 'patched' to 'resolved'
# Sun Jul 02 23:47:12 2017 steve [...] deefs.net - Fixed in 2.032 added

This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.