From a1d2c58985584116d20fa5e132137d8ff1a535f7 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Tue, 10 Feb 2015 18:18:17 +0100 Subject: [PATCH] updated for version 7.4.623 Problem: Crash with pattern: \(\)\{80000} (Dominique Pelle) Solution: When the max limit is large fall back to the old engine. --- src/regexp_nfa.c | 9 +++++---- src/version.c | 2 ++ 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src/regexp_nfa.c b/src/regexp_nfa.c index 77c787056..224535109 100644 --- a/src/regexp_nfa.c +++ b/src/regexp_nfa.c @@ -2034,9 +2034,10 @@ nfa_regpiece() } /* The engine is very inefficient (uses too many states) when the - * maximum is much larger than the minimum. Bail out if we can - * use the other engine. */ - if ((nfa_re_flags & RE_AUTO) && maxval > minval + 200) + * maximum is much larger than the minimum and when the maximum is + * large. Bail out if we can use the other engine. */ + if ((nfa_re_flags & RE_AUTO) + && (maxval > minval + 200 || maxval > 500)) return FAIL; /* Ignore previous call to nfa_regatom() */ @@ -4254,7 +4255,6 @@ state_in_list(l, state, subs) * Add "state" and possibly what follows to state list ".". * Returns "subs_arg", possibly copied into temp_subs. */ - static regsubs_T * addstate(l, state, subs_arg, pim, off) nfa_list_T *l; /* runtime state list */ @@ -4392,6 +4392,7 @@ skip_add: subs = &temp_subs; } + /* TODO: check for vim_realloc() returning NULL. */ l->t = vim_realloc(l->t, newlen * sizeof(nfa_thread_T)); l->len = newlen; } diff --git a/src/version.c b/src/version.c index 4116c7233..b52926a36 100644 --- a/src/version.c +++ b/src/version.c @@ -741,6 +741,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 623, /**/ 622, /**/