]> granicus.if.org Git - php/commitdiff
Fix CVE-2019-13224: don't allow different encodings for onig_new_deluxe()
authorStanislav Malyshev <stas@php.net>
Sun, 25 Aug 2019 06:11:45 +0000 (23:11 -0700)
committerStanislav Malyshev <stas@php.net>
Sun, 25 Aug 2019 06:11:45 +0000 (23:11 -0700)
Backport from https://github.com/kkos/oniguruma/commit/0f7f61ed1b7b697e283e37bd2d731d0bd57adb55

ext/mbstring/oniguruma/regext.c

index b1b957b40c1b374fca1406737d22d0305f8c9b72..b108e638c67e9b8314594cdc8556841fceff3ad6 100644 (file)
@@ -29,6 +29,7 @@
 
 #include "regint.h"
 
+#if 0
 static void
 conv_ext0be32(const UChar* s, const UChar* end, UChar* conv)
 {
@@ -158,6 +159,7 @@ conv_encoding(OnigEncoding from, OnigEncoding to, const UChar* s, const UChar* e
 
   return ONIGERR_NOT_SUPPORTED_ENCODING_COMBINATION;
 }
+#endif
 
 extern int
 onig_new_deluxe(regex_t** reg, const UChar* pattern, const UChar* pattern_end,
@@ -169,9 +171,7 @@ onig_new_deluxe(regex_t** reg, const UChar* pattern, const UChar* pattern_end,
   if (IS_NOT_NULL(einfo)) einfo->par = (UChar* )NULL;
 
   if (ci->pattern_enc != ci->target_enc) {
-    r = conv_encoding(ci->pattern_enc, ci->target_enc, pattern, pattern_end,
-                      &cpat, &cpat_end);
-    if (r) return r;
+    return ONIGERR_NOT_SUPPORTED_ENCODING_COMBINATION;
   }
   else {
     cpat     = (UChar* )pattern;