~ruther/guix-local

1ec0b7407f27a12adc6978d98832c41226465101 — Marius Bakke 8 years ago 4fc61da
gnu: libxfont@2: Update replacement to 2.0.3 [fixes CVE-2017-16611].

* gnu/packages/xorg.scm (libxfont2/fixed): Rename to ...
(libxfont2-2.0.3): ... this.  Make public.
[version]: New field.
[source]: Add URI and SHA256.  Remove patches.
(libxfont2)[replacement]: Change LIBXFONT2/FIXED to LIBXFONT2-2.0.3.
* gnu/packages/patches/libxfont-CVE-2017-13720.patch,
gnu/packages/patches/libxfont-CVE-2017-13722.patch: Delete files.
* gnu/local.mk (dist_patch_DATA): Remove them.
4 files changed, 12 insertions(+), 95 deletions(-)

M gnu/local.mk
D gnu/packages/patches/libxfont-CVE-2017-13720.patch
D gnu/packages/patches/libxfont-CVE-2017-13722.patch
M gnu/packages/xorg.scm
M gnu/local.mk => gnu/local.mk +0 -2
@@ 842,8 842,6 @@ dist_patch_DATA =						\
  %D%/packages/patches/libvirt-CVE-2017-1000256.patch		\
  %D%/packages/patches/libvpx-CVE-2016-2818.patch		\
  %D%/packages/patches/libxcb-python-3.5-compat.patch		\
  %D%/packages/patches/libxfont-CVE-2017-13720.patch		\
  %D%/packages/patches/libxfont-CVE-2017-13722.patch		\
  %D%/packages/patches/libxml2-CVE-2016-4658.patch		\
  %D%/packages/patches/libxml2-CVE-2016-5131.patch		\
  %D%/packages/patches/libxml2-CVE-2017-0663.patch		\

D gnu/packages/patches/libxfont-CVE-2017-13720.patch => gnu/packages/patches/libxfont-CVE-2017-13720.patch +0 -36
@@ 1,36 0,0 @@
Fix CVE-2017-13720.

Copied from upstream source repository:
<https://cgit.freedesktop.org/xorg/lib/libXfont/commit/?id=d1e670a4a8704b8708e493ab6155589bcd570608>


From d1e670a4a8704b8708e493ab6155589bcd570608 Mon Sep 17 00:00:00 2001
From: Michal Srb <msrb@suse.com>
Date: Thu, 20 Jul 2017 13:38:53 +0200
Subject: Check for end of string in PatternMatch (CVE-2017-13720)

If a pattern contains '?' character, any character in the string is skipped,
even if it is '\0'. The rest of the matching then reads invalid memory.

Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Julien Cristau <jcristau@debian.org>

diff --git a/src/fontfile/fontdir.c b/src/fontfile/fontdir.c
index 4ce2473..996b7d1 100644
--- a/src/fontfile/fontdir.c
+++ b/src/fontfile/fontdir.c
@@ -400,8 +400,10 @@ PatternMatch(char *pat, int patdashes, char *string, int stringdashes)
 		}
 	    }
 	case '?':
-	    if (*string++ == XK_minus)
+	    if ((t = *string++) == XK_minus)
 		stringdashes--;
+	    if (!t)
+		return 0;
 	    break;
 	case '\0':
 	    return (*string == '\0');
-- 
cgit v0.10.2


D gnu/packages/patches/libxfont-CVE-2017-13722.patch => gnu/packages/patches/libxfont-CVE-2017-13722.patch +0 -53
@@ 1,53 0,0 @@
Fix CVE-2017-13722.

Copied from upstream source repository:
<https://cgit.freedesktop.org/xorg/lib/libXfont/commit/?id=672bb944311392e2415b39c0d63b1e1902905bcd>

From 672bb944311392e2415b39c0d63b1e1902905bcd Mon Sep 17 00:00:00 2001
From: Michal Srb <msrb@suse.com>
Date: Thu, 20 Jul 2017 17:05:23 +0200
Subject: pcfGetProperties: Check string boundaries (CVE-2017-13722)

Without the checks a malformed PCF file can cause the library to make
atom from random heap memory that was behind the `strings` buffer.
This may crash the process or leak information.

Signed-off-by: Julien Cristau <jcristau@debian.org>

diff --git a/src/bitmap/pcfread.c b/src/bitmap/pcfread.c
index dab1c44..ae34c28 100644
--- a/src/bitmap/pcfread.c
+++ b/src/bitmap/pcfread.c
@@ -45,6 +45,7 @@ from The Open Group.
 
 #include <stdarg.h>
 #include <stdint.h>
+#include <string.h>
 
 void
 pcfError(const char* message, ...)
@@ -311,11 +312,19 @@ pcfGetProperties(FontInfoPtr pFontInfo, FontFilePtr file,
     if (IS_EOF(file)) goto Bail;
     position += string_size;
     for (i = 0; i < nprops; i++) {
+	if (props[i].name >= string_size) {
+	    pcfError("pcfGetProperties(): String starts out of bounds (%ld/%d)\n", props[i].name, string_size);
+	    goto Bail;
+	}
 	props[i].name = MakeAtom(strings + props[i].name,
-				 strlen(strings + props[i].name), TRUE);
+				 strnlen(strings + props[i].name, string_size - props[i].name), TRUE);
 	if (isStringProp[i]) {
+	    if (props[i].value >= string_size) {
+		pcfError("pcfGetProperties(): String starts out of bounds (%ld/%d)\n", props[i].value, string_size);
+		goto Bail;
+	    }
 	    props[i].value = MakeAtom(strings + props[i].value,
-				      strlen(strings + props[i].value), TRUE);
+				      strnlen(strings + props[i].value, string_size - props[i].value), TRUE);
 	}
     }
     free(strings);
-- 
cgit v0.10.2


M gnu/packages/xorg.scm => gnu/packages/xorg.scm +12 -4
@@ 4892,7 4892,7 @@ new API's in libXft, or the legacy API's in libX11.")
  (package
    (inherit libxfont)
    (version "2.0.1")
    (replacement libxfont2/fixed)
    (replacement libxfont2-2.0.3)
    (source (origin
              (method url-fetch)
              (uri (string-append "mirror://xorg/individual/lib/libXfont2-"


@@ 4901,14 4901,22 @@ new API's in libXft, or the legacy API's in libX11.")
               (base32
                "0znvwk36nhmyqpmhbm9mzisgixp1mp5qkfald8x1n5yxbm3vpyz9"))))))

(define libxfont2/fixed
;; Fixes the following security vulnerabilities:
;; https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-13720
;; https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-13722
;; https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-16612
(define-public libxfont2-2.0.3
  (package
    (inherit libxfont2)
    (version "2.0.3")
    (source
     (origin
       (inherit (package-source libxfont2))
       (patches (search-patches "libxfont-CVE-2017-13720.patch"
                                "libxfont-CVE-2017-13722.patch"))))))
       (uri (string-append "mirror://xorg/individual/lib/libXfont2-"
                           version ".tar.bz2"))
       (sha256
        (base32
         "0klwmimmhm3axpj8pwn5l41lbggh47r5aazhw63zxkbwfgyvg2hf"))))))

(define-public libxi
  (package