[gs-bugs] [Bug 697795] - MuPDF - pdf_save_document : Regression in 1.11 ?

[b***@artifex.com]

http://bugs.ghostscript.com/show_bug.cgi?id=697795

Bug ID: 697795
Summary: pdf_save_document : Regression in 1.11 ?
Product: MuPDF
Version: master
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: P4
Component: mupdf
Assignee: mupdf-***@artifex.com
Reporter: ***@gmail.com
QA Contact: gs-***@ghostscript.com
Word Size: ---

Created attachment 13615
--> http://bugs.ghostscript.com/attachment.cgi?id=13615&action=edit
Generated with MuPDF 1.10a (good)

I just upgraded my apps with MuPDF 1.11.
The upgrade only required small chnages in my app ( due to two missing
functions: pdf_drop_page() and fz_write() )
and everything worked accordling to my old test-suite; just one serious bug:
the pdf_save_document() now generates wrong (corrupted) PDFs.

------------
When I save (in incremental mode) a linearized PDF, the resulting PDF is
corrupted.
------------

Here you can find two files, the first generated with 1.10a, and the second
with 1.11

The only difference is on the line before the last (startxref); this value in
Book-1.11 is huge and simply wrong.

I repeated these tests with different files: everytime I use a linearized-PDF ,
result is corrupted.

[b***@artifex.com]

http://bugs.ghostscript.com/show_bug.cgi?id=697795

--- Comment #1 from Ardo <***@gmail.com> ---
Created attachment 13616
--> http://bugs.ghostscript.com/attachment.cgi?id=13616&action=edit
Generated withMuPdf 1.11 (corrupted)

[b***@artifex.com]

http://bugs.ghostscript.com/show_bug.cgi?id=697795

Ardo <***@gmail.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Blocks| |698375

[b***@artifex.com]

http://bugs.ghostscript.com/show_bug.cgi?id=697795

Robin Watts <***@artifex.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|UNCONFIRMED |RESOLVED
CC| |***@artifex.com

--- Comment #2 from Robin Watts <***@artifex.com> ---
Fixed as a result of:

commit f595e889b91a674eb94db7ca4d832da54f5194cd
Author: Tor Andersson <***@artifex.com>
Date: Thu Aug 17 15:01:09 2017 +0200

Use int64_t for public file API offsets.

Don't mess with conditional compilation with LARGEFILE -- always expose
64-bit file offsets in our public API.

Thanks for the report.