From 0b0c63bbd82f98c958889534944da0d12164487d Mon Sep 17 00:00:00 2001 From: Matthew Vernon Date: Sat, 24 Feb 2018 12:10:59 +0000 Subject: [PATCH] Revert "CVE-2017-7186 fix (upstream rev 670)" This reverts commit 8a44b106f405e278f4fbbcd1e8b2744cd9afbc57. Do this as we're now incorporating a new upstream version that includes this fix, and there's a merge conflict in the testoutput files. --- src/pcre2_internal.h | 15 +-------------- src/pcre2_ucd.c | 14 -------------- testdata/testinput12 | 3 --- testdata/testoutput12-16 | 6 ------ testdata/testoutput12-32 | 4 ---- 5 files changed, 1 insertion(+), 41 deletions(-) diff --git a/src/pcre2_internal.h b/src/pcre2_internal.h index 7fb374d..5690870 100644 --- a/src/pcre2_internal.h +++ b/src/pcre2_internal.h @@ -1794,17 +1794,10 @@ typedef struct { /* UCD access macros */ #define UCD_BLOCK_SIZE 128 -#define REAL_GET_UCD(ch) (PRIV(ucd_records) + \ +#define GET_UCD(ch) (PRIV(ucd_records) + \ PRIV(ucd_stage2)[PRIV(ucd_stage1)[(int)(ch) / UCD_BLOCK_SIZE] * \ UCD_BLOCK_SIZE + (int)(ch) % UCD_BLOCK_SIZE]) -#if PCRE2_CODE_UNIT_WIDTH == 32 -#define GET_UCD(ch) ((ch > MAX_UTF_CODE_POINT)? \ - PRIV(dummy_ucd_record) : REAL_GET_UCD(ch)) -#else -#define GET_UCD(ch) REAL_GET_UCD(ch) -#endif - #define UCD_CHARTYPE(ch) GET_UCD(ch)->chartype #define UCD_SCRIPT(ch) GET_UCD(ch)->script #define UCD_CATEGORY(ch) PRIV(ucp_gentype)[UCD_CHARTYPE(ch)] @@ -1861,9 +1854,6 @@ extern const uint8_t PRIV(utf8_table4)[]; #define _pcre2_default_compile_context PCRE2_SUFFIX(_pcre2_default_compile_context_) #define _pcre2_default_match_context PCRE2_SUFFIX(_pcre2_default_match_context_) #define _pcre2_default_tables PCRE2_SUFFIX(_pcre2_default_tables_) -#if PCRE2_CODE_UNIT_WIDTH == 32 -#define _pcre2_dummy_ucd_record PCRE2_SUFFIX(_pcre2_dummy_ucd_record_) -#endif #define _pcre2_hspace_list PCRE2_SUFFIX(_pcre2_hspace_list_) #define _pcre2_vspace_list PCRE2_SUFFIX(_pcre2_vspace_list_) #define _pcre2_ucd_caseless_sets PCRE2_SUFFIX(_pcre2_ucd_caseless_sets_) @@ -1888,9 +1878,6 @@ extern const uint32_t PRIV(hspace_list)[]; extern const uint32_t PRIV(vspace_list)[]; extern const uint32_t PRIV(ucd_caseless_sets)[]; extern const ucd_record PRIV(ucd_records)[]; -#if PCRE2_CODE_UNIT_WIDTH == 32 -extern const ucd_record PRIV(dummy_ucd_record)[]; -#endif extern const uint8_t PRIV(ucd_stage1)[]; extern const uint16_t PRIV(ucd_stage2)[]; extern const uint32_t PRIV(ucp_gbtable)[]; diff --git a/src/pcre2_ucd.c b/src/pcre2_ucd.c index 56aa29d..116f537 100644 --- a/src/pcre2_ucd.c +++ b/src/pcre2_ucd.c @@ -41,20 +41,6 @@ const uint32_t PRIV(ucd_caseless_sets)[] = {0}; const char *PRIV(unicode_version) = "8.0.0"; -/* If the 32-bit library is run in non-32-bit mode, character values -greater than 0x10ffff may be encountered. For these we set up a -special record. */ - -#if PCRE2_CODE_UNIT_WIDTH == 32 -const ucd_record PRIV(dummy_ucd_record)[] = {{ - ucp_Common, /* script */ - ucp_Cn, /* type unassigned */ - ucp_gbOther, /* grapheme break property */ - 0, /* case set */ - 0, /* other case */ - }}; -#endif - /* When recompiling tables with a new Unicode version, please check the types in this structure definition from pcre2_internal.h (the actual field names will be different): diff --git a/testdata/testinput12 b/testdata/testinput12 index 86f2c74..14a7715 100644 --- a/testdata/testinput12 +++ b/testdata/testinput12 @@ -343,7 +343,4 @@ /./utf \x{110000} -/\pP/ucp - \x{7fffffff}\=no_jit - # End of testinput12 diff --git a/testdata/testoutput12-16 b/testdata/testoutput12-16 index b19f6c4..383a032 100644 --- a/testdata/testoutput12-16 +++ b/testdata/testoutput12-16 @@ -1367,10 +1367,4 @@ Subject length lower bound = 2 \x{110000} ** Failed: character \x{110000} is greater than 0x10ffff and so cannot be converted to UTF-16 -/\pP/ucp - \x{7fffffff}\=no_jit -** Character \x{7fffffff} is greater than 0xffff and UTF-16 mode is not enabled. -** Truncation will probably give the wrong result. -No match - # End of testinput12 diff --git a/testdata/testoutput12-32 b/testdata/testoutput12-32 index 220f4f1..95f1834 100644 --- a/testdata/testoutput12-32 +++ b/testdata/testoutput12-32 @@ -1361,8 +1361,4 @@ Subject length lower bound = 2 \x{110000} Failed: error -28: UTF-32 error: code points greater than 0x10ffff are not defined at offset 0 -/\pP/ucp - \x{7fffffff}\=no_jit -No match - # End of testinput12 -- 2.30.2