CVS commit: pkgsrc/textproc/libxslt

["Tim Zingelman" <tez%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   tez
Date:           Tue May 23 23:37:01 UTC 2017

Modified Files:
        pkgsrc/textproc/libxslt: Makefile distinfo
Added Files:
        pkgsrc/textproc/libxslt/patches: patch-CVE-2017-5029

Log Message:
Add patch for CVE-2017-5029


To generate a diff of this commit:
cvs rdiff -u -r1.104 -r1.105 pkgsrc/textproc/libxslt/Makefile
cvs rdiff -u -r1.59 -r1.60 pkgsrc/textproc/libxslt/distinfo
cvs rdiff -u -r0 -r1.1 pkgsrc/textproc/libxslt/patches/patch-CVE-2017-5029

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/textproc/libxslt/Makefile
diff -u pkgsrc/textproc/libxslt/Makefile:1.104 pkgsrc/textproc/libxslt/Makefile:1.105
--- pkgsrc/textproc/libxslt/Makefile:1.104      Sat Jul  9 06:39:06 2016
+++ pkgsrc/textproc/libxslt/Makefile    Tue May 23 23:37:01 2017
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.104 2016/07/09 06:39:06 wiz Exp $
+# $NetBSD: Makefile,v 1.105 2017/05/23 23:37:01 tez Exp $
 
-PKGREVISION= 1
+PKGREVISION= 2
 .include "Makefile.common"
 
 BUILD_DEPENDS+=        docbook-xml-[0-9]*:../../textproc/docbook-xml

Index: pkgsrc/textproc/libxslt/distinfo
diff -u pkgsrc/textproc/libxslt/distinfo:1.59 pkgsrc/textproc/libxslt/distinfo:1.60
--- pkgsrc/textproc/libxslt/distinfo:1.59       Tue May 16 13:51:50 2017
+++ pkgsrc/textproc/libxslt/distinfo    Tue May 23 23:37:01 2017
@@ -1,8 +1,9 @@
-$NetBSD: distinfo,v 1.59 2017/05/16 13:51:50 dholland Exp $
+$NetBSD: distinfo,v 1.60 2017/05/23 23:37:01 tez Exp $
 
 SHA1 (libxslt-1.1.29.tar.gz) = edcaeabb3555ae44853bdc406ee9521fb65c620d
 RMD160 (libxslt-1.1.29.tar.gz) = 40a5202b3967db6b01cd6c0e722a9e459cac7960
 SHA512 (libxslt-1.1.29.tar.gz) = a1ce555a74a9dabe65e8f64bb66e27e77760fd76940d88f2d59f58dd63ca73c8ae59f3fcbd8e76c8f92ff992fb0c09328528c20ea38ccac83e63252106bf5f31
 Size (libxslt-1.1.29.tar.gz) = 3428524 bytes
+SHA1 (patch-CVE-2017-5029) = 98658fa048e440ef72460d4e082c0fd282154aa1
 SHA1 (patch-ae) = 40ce3940a93b6a2dc804f62676909d3313e0ea52
 SHA1 (patch-ah) = f92809ab34e53962ec8b62e50ded7a27c89f10f8

Added files:

Index: pkgsrc/textproc/libxslt/patches/patch-CVE-2017-5029
diff -u /dev/null pkgsrc/textproc/libxslt/patches/patch-CVE-2017-5029:1.1
--- /dev/null   Tue May 23 23:37:01 2017
+++ pkgsrc/textproc/libxslt/patches/patch-CVE-2017-5029 Tue May 23 23:37:01 2017
@@ -0,0 +1,58 @@
+$NetBSD: patch-CVE-2017-5029,v 1.1 2017/05/23 23:37:01 tez Exp $
+
+Patch for CVE-2017-5029 from:
+ https://git.gnome.org/browse/libxslt/commit/?id=08ab2774b870de1c7b5a48693df75e8154addae5
+
+
+--- libxslt/xsltInternals.h.orig       2017-05-23 23:14:05.625798800 +0000
++++ libxslt/xsltInternals.h
+@@ -1754,8 +1754,8 @@ struct _xsltTransformContext {
+      * Speed optimization when coalescing text nodes
+      */
+     const xmlChar  *lasttext;         /* last text node content */
+-    unsigned int    lasttsize;                /* last text node size */
+-    unsigned int    lasttuse;         /* last text node use */
++    int                   lasttsize;          /* last text node size */
++    int                   lasttuse;           /* last text node use */
+     /*
+      * Per Context Debugging
+      */
+
+--- libxslt/transform.c.orig   2017-05-23 23:14:23.480987400 +0000
++++ libxslt/transform.c
+@@ -816,13 +816,32 @@ xsltAddTextString(xsltTransformContextPt
+         return(target);
+ 
+     if (ctxt->lasttext == target->content) {
++      int minSize;
+ 
+-      if (ctxt->lasttuse + len >= ctxt->lasttsize) {
++      /* Check for integer overflow accounting for NUL terminator. */
++      if (len >= INT_MAX - ctxt->lasttuse) {
++          xsltTransformError(ctxt, NULL, target,
++              "xsltCopyText: text allocation failed\n");
++          return(NULL);
++      }
++      minSize = ctxt->lasttuse + len + 1;
++
++      if (ctxt->lasttsize < minSize) {
+           xmlChar *newbuf;
+           int size;
++          int extra;
++
++          /* Double buffer size but increase by at least 100 bytes. */
++          extra = minSize < 100 ? 100 : minSize;
++
++          /* Check for integer overflow. */
++          if (extra > INT_MAX - ctxt->lasttsize) {
++              size = INT_MAX;
++          }
++          else {
++              size = ctxt->lasttsize + extra;
++          }
+ 
+-          size = ctxt->lasttsize + len + 100;
+-          size *= 2;
+           newbuf = (xmlChar *) xmlRealloc(target->content,size);
+           if (newbuf == NULL) {
+               xsltTransformError(ctxt, NULL, target,