From: Richard M. Stallman <rms@gnu.org>
Date: Fri, 12 Apr 1996 22:57:28 +0000 (+0000)
Subject: (MAX_FAILURE_ITEMS): Use 5 instead of actual number of regs.
X-Git-Tag: emacs-19.34~849
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=ffd76827b0eda72f38f6972d49b38bdd6a01602a;p=emacs.git

(MAX_FAILURE_ITEMS): Use 5 instead of actual number of regs.
---

diff --git a/src/regex.c b/src/regex.c
index 7128c4e87c1..e6d04a57df5 100644
--- a/src/regex.c
+++ b/src/regex.c
@@ -1004,7 +1004,7 @@ static const char *re_error_msgid[] =
 #endif
 
 /* Roughly the maximum number of failure points on the stack.  Would be
-   exactly that if always used MAX_FAILURE_SPACE each time we failed.
+   exactly that if always used MAX_FAILURE_ITEMS items each time we failed.
    This is a variable only so users of regex can assign to it; we never
    change it ourselves.  */
 #if defined (MATCH_MAY_ALLOCATE)
@@ -1221,7 +1221,10 @@ typedef struct
 #endif
 
 /* We push at most this many items on the stack.  */
-#define MAX_FAILURE_ITEMS ((num_regs - 1) * NUM_REG_ITEMS + NUM_NONREG_ITEMS)
+/* We used to use (num_regs - 1), which is the number of registers
+   this regexp will save; but that was changed to 5
+   to avoid stack overflow for a regexp with lots of parens.  */
+#define MAX_FAILURE_ITEMS (5 * NUM_REG_ITEMS + NUM_NONREG_ITEMS)
 
 /* We actually push this many items.  */
 #define NUM_FAILURE_ITEMS				\