From d11fceab151cd0410645f81eb7444af4388470c3 Mon Sep 17 00:00:00 2001 From: Anatol Belski Date: Thu, 28 Sep 2017 15:40:49 +0200 Subject: [PATCH] Apply upstream patch for CVE-2016-1283 Fix bug #75207, see also https://bugzilla.redhat.com/show_bug.cgi?id=1295385 https://vcs.pcre.org/pcre?view=revision&revision=1636 --- ext/pcre/pcrelib/pcre_compile.c | 7 ++++++- ext/pcre/tests/bug75207.phpt | 10 ++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) create mode 100644 ext/pcre/tests/bug75207.phpt diff --git a/ext/pcre/pcrelib/pcre_compile.c b/ext/pcre/pcrelib/pcre_compile.c index 11a9d26ff6a..c7827745c86 100644 --- a/ext/pcre/pcrelib/pcre_compile.c +++ b/ext/pcre/pcrelib/pcre_compile.c @@ -7272,7 +7272,12 @@ for (;; ptr++) so far in order to get the number. If the name is not found, leave the value of recno as 0 for a forward reference. */ - else + /* This patch (removing "else") fixes a problem when a reference is + to multiple identically named nested groups from within the nest. + Once again, it is not the "proper" fix, and it results in an + over-allocation of memory. */ + + /* else */ { ng = cd->named_groups; for (i = 0; i < cd->names_found; i++, ng++) diff --git a/ext/pcre/tests/bug75207.phpt b/ext/pcre/tests/bug75207.phpt new file mode 100644 index 00000000000..dea13a44cac --- /dev/null +++ b/ext/pcre/tests/bug75207.phpt @@ -0,0 +1,10 @@ +--TEST-- +CVE-2016-1283, see bug #75207 +--FILE-- + +==DONE== +--EXPECTF-- +Warning: preg_match(): Compilation failed: unmatched parentheses at offset %d in %s on line %d +==DONE==