Blame texlive-20200327-poppler-0.84.patch

75e825d
diff -up texlive-base-20200327/source/texk/web2c/pdftexdir/pdftoepdf.cc.poppler-0.84 texlive-base-20200327/source/texk/web2c/pdftexdir/pdftoepdf.cc
75e825d
--- texlive-base-20200327/source/texk/web2c/pdftexdir/pdftoepdf.cc.poppler-0.84	2020-04-20 14:38:05.301181578 -0400
75e825d
+++ texlive-base-20200327/source/texk/web2c/pdftexdir/pdftoepdf.cc	2020-04-20 14:38:05.314181306 -0400
75e825d
@@ -26,6 +26,15 @@ The poppler should be 0.59.0 or newer ve
75e825d
 POPPLER_VERSION should be defined.
75e825d
 */
75e825d
 
75e825d
+#ifdef POPPLER_VERSION
75e825d
+#include <poppler-config.h>
75e825d
+#define xpdfVersion POPPLER_VERSION
75e825d
+#define xpdfString "poppler"
75e825d
+#else
75e825d
+#include <xpdf/config.h>        /* just to get the xpdf version */
75e825d
+#define xpdfString "xpdf"
75e825d
+#endif
75e825d
+
75e825d
 /* Do this early in order to avoid a conflict between
75e825d
    MINGW32 <rpcndr.h> defining 'boolean' as 'unsigned char' and
75e825d
    <kpathsea/types.h> defining Pascal's boolean as 'int'.
75e825d
@@ -292,7 +301,7 @@ static void copyDictEntry(Object * obj,
75e825d
     Object obj1;
75e825d
     copyName(obj->dictGetKey(i));
75e825d
     pdf_puts(" ");
75e825d
-    obj1 = obj->dictGetValNF(i);
75e825d
+    obj1 = obj->dictGetValNF(i).copy();
75e825d
     copyObject(&obj1);
75e825d
     pdf_puts("\n");
75e825d
 }
75e825d
@@ -351,7 +360,7 @@ static void copyProcSet(Object * obj)
75e825d
                     obj->getTypeName());
75e825d
     pdf_puts("/ProcSet [ ");
75e825d
     for (i = 0, l = obj->arrayGetLength(); i < l; ++i) {
75e825d
-        procset = obj->arrayGetNF(i);
75e825d
+        procset = obj->arrayGetNF(i).copy();
75e825d
         if (!procset.isName())
75e825d
             pdftex_fail("PDF inclusion: invalid ProcSet entry type <%s>",
75e825d
                         procset.getTypeName());
75e825d
@@ -406,7 +415,7 @@ static void copyFont(const char *tag, Ob
75e825d
     if (fontdict.isDict()) {
75e825d
         subtype = fontdict.dictLookup("Subtype");
75e825d
         basefont = fontdict.dictLookup("BaseFont");
75e825d
-        fontdescRef = fontdict.dictLookupNF("FontDescriptor");
75e825d
+        fontdescRef = fontdict.dictLookupNF("FontDescriptor").copy();
75e825d
         if (fontdescRef.isRef()) {
75e825d
             fontdesc = fontdescRef.fetch(xref);
75e825d
         }
75e825d
@@ -452,7 +461,7 @@ static void copyFontResources(Object * o
75e825d
                     obj->getTypeName());
75e825d
     pdf_puts("/Font << ");
75e825d
     for (i = 0, l = obj->dictGetLength(); i < l; ++i) {
75e825d
-        fontRef = obj->dictGetValNF(i);
75e825d
+        fontRef = obj->dictGetValNF(i).copy();
75e825d
         if (fontRef.isRef())
75e825d
             copyFont(obj->dictGetKey(i), &fontRef);
75e825d
         else if (fontRef.isDict()) {   // some programs generate pdf with embedded font object
75e825d
@@ -595,7 +604,7 @@ static void copyObject(Object * obj)
75e825d
     } else if (obj->isArray()) {
75e825d
         pdf_puts("[");
75e825d
         for (i = 0, l = obj->arrayGetLength(); i < l; ++i) {
75e825d
-            obj1 = obj->arrayGetNF(i);
75e825d
+            obj1 = obj->arrayGetNF(i).copy();
75e825d
             if (!obj1.isName())
75e825d
                 pdf_puts(" ");
75e825d
             copyObject(&obj1);
75e825d
@@ -723,7 +732,7 @@ read_pdf_info(char *image_name, char *pa
75e825d
 #endif
75e825d
     // initialize
75e825d
     if (!isInit) {
75e825d
-        globalParams = new GlobalParams();
75e825d
+        globalParams = std::unique_ptr<GlobalParams>(new GlobalParams());
75e825d
         globalParams->setErrQuiet(false);
75e825d
         isInit = true;
75e825d
     }
75e825d
@@ -761,7 +770,7 @@ read_pdf_info(char *image_name, char *pa
75e825d
         if (link == 0 || !link->isOk())
75e825d
             pdftex_fail("PDF inclusion: invalid destination <%s>", page_name);
75e825d
         Ref ref = link->getPageRef();
75e825d
-        page_num = pdf_doc->doc->getCatalog()->findPage(ref.num, ref.gen);
75e825d
+        page_num = pdf_doc->doc->getCatalog()->findPage(ref);
75e825d
         if (page_num == 0)
75e825d
             pdftex_fail("PDF inclusion: destination is not a page <%s>",
75e825d
                         page_name);
75e825d
@@ -921,13 +930,13 @@ void write_epdf(void)
75e825d
     pdf_puts(stripzeros(s));
75e825d
 
75e825d
     // Metadata validity check (as a stream it must be indirect)
75e825d
-    dictObj = pageDict->lookupNF("Metadata");
75e825d
+    dictObj = pageDict->lookupNF("Metadata").copy();
75e825d
     if (!dictObj.isNull() && !dictObj.isRef())
75e825d
         pdftex_warn("PDF inclusion: /Metadata must be indirect object");
75e825d
 
75e825d
     // copy selected items in Page dictionary except Resources & Group
75e825d
     for (i = 0; pageDictKeys[i] != NULL; i++) {
75e825d
-        dictObj = pageDict->lookupNF(pageDictKeys[i]);
75e825d
+        dictObj = pageDict->lookupNF(pageDictKeys[i]).copy();
75e825d
         if (!dictObj.isNull()) {
75e825d
             pdf_newline();
75e825d
             pdf_printf("/%s ", pageDictKeys[i]);
75e825d
@@ -936,7 +945,7 @@ void write_epdf(void)
75e825d
     } 
75e825d
 
75e825d
     // handle page group
75e825d
-    dictObj = pageDict->lookupNF("Group");
75e825d
+    dictObj = pageDict->lookupNF("Group").copy();
75e825d
     if (!dictObj.isNull()) {
75e825d
         if (pdfpagegroupval == 0) { 
75e825d
             // another pdf with page group was included earlier on the
75e825d
@@ -978,7 +987,7 @@ The changes below seem to work fine.
75e825d
             l = dic1.getLength();
75e825d
             for (i = 0; i < l; i++) {
75e825d
                 groupDict.dictAdd(dic1.getKey(i),
75e825d
-                                  dic1.getValNF(i));
75e825d
+                                  dic1.getValNF(i).copy());
75e825d
             }
75e825d
 // end modification
75e825d
             pdf_printf("/Group %ld 0 R\n", (long)pdfpagegroupval);
75e825d
@@ -1108,6 +1117,6 @@ void epdf_check_mem()
75e825d
             delete_document(p);
75e825d
         }
75e825d
         // see above for globalParams
75e825d
-        delete globalParams;
75e825d
+        globalParams.reset();
75e825d
     }
75e825d
 }
75e825d
diff -up texlive-base-20200327/source/texk/web2c/pdftexdir/pdftosrc.cc.poppler-0.84 texlive-base-20200327/source/texk/web2c/pdftexdir/pdftosrc.cc
75e825d
--- texlive-base-20200327/source/texk/web2c/pdftexdir/pdftosrc.cc.poppler-0.84	2020-04-20 14:38:05.301181578 -0400
75e825d
+++ texlive-base-20200327/source/texk/web2c/pdftexdir/pdftosrc.cc	2020-04-20 14:38:05.314181306 -0400
75e825d
@@ -24,6 +24,15 @@ by Arch Linux. The poppler should be 0.5
75e825d
 POPPLER_VERSION should be defined.
75e825d
 */
75e825d
 
75e825d
+#ifdef POPPLER_VERSION
75e825d
+#include <poppler-config.h>
75e825d
+#define xpdfVersion POPPLER_VERSION
75e825d
+#define xpdfString "poppler"
75e825d
+#else
75e825d
+#include <xpdf/config.h>        /* just to get the xpdf version */
75e825d
+#define xpdfString "xpdf"
75e825d
+#endif
75e825d
+
75e825d
 #include <w2c/config.h>
75e825d
 
75e825d
 #include <stdlib.h>
75e825d
@@ -79,7 +88,7 @@ int main(int argc, char *argv[])
75e825d
         exit(1);
75e825d
     }
75e825d
     fileName = new GString(argv[1]);
75e825d
-    globalParams = new GlobalParams();
75e825d
+    globalParams = std::unique_ptr<GlobalParams>(new GlobalParams());
75e825d
     doc = new PDFDoc(fileName);
75e825d
     if (!doc->isOk()) {
75e825d
         fprintf(stderr, "Invalid PDF file\n");
75e825d
@@ -100,7 +109,7 @@ int main(int argc, char *argv[])
75e825d
     if (objnum == 0) {
75e825d
         srcStream = catalogDict.dictLookup("SourceObject");
75e825d
         static char const_SourceFile[] = "SourceFile";
75e825d
-        if (!srcStream.isStream(const_SourceFile)) {
75e825d
+        if (!(srcStream.isStream() && srcStream.getDict()->is(const_SourceFile))) {
75e825d
             fprintf(stderr, "No SourceObject found\n");
75e825d
             exit(1);
75e825d
         }
75e825d
@@ -156,7 +165,6 @@ int main(int argc, char *argv[])
75e825d
                         (e->type == xrefEntryFree ? "f" : "n"));
75e825d
             else {              // e->offset is the object number of the object stream
75e825d
                 Stream *str;
75e825d
-                Lexer *lexer;
75e825d
                 Parser *parser;
75e825d
                 Object objStr, obj1, obj2;
75e825d
                 int nObjects, first, n;
75e825d
@@ -174,8 +182,7 @@ int main(int argc, char *argv[])
75e825d
                 // parse the header: object numbers and offsets
75e825d
                 objStr.streamReset();
75e825d
                 str = new EmbedStream(objStr.getStream(), Object(objNull), true, first);
75e825d
-                lexer = new Lexer(xref, str);
75e825d
-                parser = new Parser(xref, lexer, false);
75e825d
+                parser = new Parser(xref, str, false);
75e825d
                 for (n = 0; n < nObjects; ++n) {
75e825d
                     obj1 = parser->getObj();
75e825d
                     obj2 = parser->getObj();
75e825d
@@ -207,5 +214,5 @@ int main(int argc, char *argv[])
75e825d
         fprintf(stderr, "Cross-reference table extracted to %s\n", outname);
75e825d
     fclose(outfile);
75e825d
     delete doc;
75e825d
-    delete globalParams;
75e825d
+    globalParams.reset();
75e825d
 }
75e825d
diff -up texlive-base-20200327/source/texk/web2c/pdftexdir/utils.c.poppler-0.84 texlive-base-20200327/source/texk/web2c/pdftexdir/utils.c
75e825d
--- texlive-base-20200327/source/texk/web2c/pdftexdir/utils.c.poppler-0.84	2019-12-29 19:37:32.000000000 -0500
75e825d
+++ texlive-base-20200327/source/texk/web2c/pdftexdir/utils.c	2020-04-20 14:39:39.355214866 -0400
75e825d
@@ -32,14 +32,6 @@ with this program.  If not, see 
75e825d
 #include <zlib.h>
75e825d
 #include "ptexlib.h"
75e825d
 #include <png.h>
75e825d
-#ifdef POPPLER_VERSION
75e825d
-/* POPPLER_VERSION should be a proper version string */
75e825d
-#define xpdfVersion POPPLER_VERSION
75e825d
-#define xpdfString "poppler"
75e825d
-#else
75e825d
-#include <xpdf/config.h>        /* just to get the xpdf version */
75e825d
-#define xpdfString "xpdf"
75e825d
-#endif
75e825d
 
75e825d
 #define check_nprintf(size_get, size_want) \
75e825d
     if ((unsigned)(size_get) >= (unsigned)(size_want)) \
75e825d
@@ -977,12 +969,10 @@ void initversionstring(char **versions)
75e825d
 {
75e825d
     const_string fmt =
75e825d
                     "Compiled with libpng %s; using libpng %s\n"
75e825d
-                    "Compiled with zlib %s; using zlib %s\n"
75e825d
-                    "Compiled with %s version %s\n";
75e825d
+                    "Compiled with zlib %s; using zlib %s\n";
75e825d
     size_t len = strlen(fmt)
75e825d
                     + strlen(PNG_LIBPNG_VER_STRING) + strlen(png_libpng_ver)
75e825d
                     + strlen(ZLIB_VERSION) + strlen(zlib_version)
75e825d
-                    + strlen(xpdfString) + strlen(xpdfVersion)
75e825d
                     + 1;
75e825d
 
75e825d
     /* len will be more than enough, because of the placeholder chars in fmt
75e825d
@@ -990,7 +980,7 @@ void initversionstring(char **versions)
75e825d
     *versions = xmalloc(len);
75e825d
     sprintf(*versions, fmt,
75e825d
                     PNG_LIBPNG_VER_STRING, png_libpng_ver,
75e825d
-                    ZLIB_VERSION, zlib_version, xpdfString, xpdfVersion);
75e825d
+                    ZLIB_VERSION, zlib_version);
75e825d
 }
75e825d
 
75e825d
 
75e825d
diff -up texlive-base-20200327/source/texk/web2c/xetexdir/XeTeX_ext.c.poppler-0.84 texlive-base-20200327/source/texk/web2c/xetexdir/XeTeX_ext.c
75e825d
--- texlive-base-20200327/source/texk/web2c/xetexdir/XeTeX_ext.c.poppler-0.84	2019-12-30 16:00:39.000000000 -0500
75e825d
+++ texlive-base-20200327/source/texk/web2c/xetexdir/XeTeX_ext.c	2020-04-20 14:40:44.534851910 -0400
75e825d
@@ -38,10 +38,6 @@ authorization from the copyright holders
75e825d
 
75e825d
 #include <w2c/config.h>
75e825d
 
75e825d
-#ifndef POPPLER_VERSION
75e825d
-#include <poppler-config.h>
75e825d
-#endif
75e825d
-
75e825d
 #include <png.h>
75e825d
 #include <zlib.h>
75e825d
 #include <graphite2/Font.h>
75e825d
@@ -170,7 +166,6 @@ void initversionstring(char **versions)
75e825d
         "Compiled with Graphite2 version %d.%d.%d; using %d.%d.%d\n"
75e825d
         "Compiled with HarfBuzz version %s; using %s\n"
75e825d
         "Compiled with libpng version %s; using %s\n"
75e825d
-        "Compiled with poppler version %s\n"
75e825d
 #ifdef XETEX_MAC
75e825d
         "Using Mac OS X Core Text and Cocoa frameworks\n"
75e825d
 #else
75e825d
@@ -187,7 +182,6 @@ void initversionstring(char **versions)
75e825d
             + strlen(hb_version_string())
75e825d
             + strlen(PNG_LIBPNG_VER_STRING)
75e825d
             + strlen(png_libpng_ver)
75e825d
-            + strlen(POPPLER_VERSION)
75e825d
 #ifndef XETEX_MAC
75e825d
             + 6 * 3 /* for fontconfig version #s (won't really need 3 digits per field!) */
75e825d
 #endif
75e825d
@@ -217,7 +211,7 @@ void initversionstring(char **versions)
75e825d
         GR2_VERSION_MAJOR, GR2_VERSION_MINOR, GR2_VERSION_BUGFIX,
75e825d
         grMajor, grMinor, grBugfix,
75e825d
         HB_VERSION_STRING, hb_version_string(),
75e825d
-        PNG_LIBPNG_VER_STRING, png_libpng_ver, POPPLER_VERSION
75e825d
+        PNG_LIBPNG_VER_STRING, png_libpng_ver
75e825d
 #ifndef XETEX_MAC
75e825d
         ,
75e825d
         FC_VERSION / 10000, (FC_VERSION % 10000) / 100, FC_VERSION % 100,