summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog10
-rw-r--r--posix/regcomp.c2
2 files changed, 12 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 3dd3cf06fb..41247ecc92 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2014-07-13 Jim Meyering <meyering@fb.com>
+
+ [BZ 17150]
+ regex: don't deref NULL upon heap allocation failure
+ * posix/regcomp.c: (parse_dup_op): Handle duplicate_tree
+ failure in one more place.
+ To trigger the segfault, configure grep -with-included-regex,
+ build it, and run these commands:
+ ( ulimit -v 300000; echo a|src/grep -E a+++++++++++++++++++++ )
+
2014-07-13 Andreas Schwab <schwab@linux-m68k.org>
* sysdeps/m68k/m680x0/fpu/libm-test-ulps: Update.
diff --git a/posix/regcomp.c b/posix/regcomp.c
index 076eca3e7c..8f2747b3c9 100644
--- a/posix/regcomp.c
+++ b/posix/regcomp.c
@@ -2582,6 +2582,8 @@ parse_dup_op (bin_tree_t *elem, re_string_t *regexp, re_dfa_t *dfa,
/* Duplicate ELEM before it is marked optional. */
elem = duplicate_tree (elem, dfa);
+ if (BE (elem == NULL, 0))
+ goto parse_dup_op_espace;
old_tree = tree;
}
else