From: Bram Moolenaar Date: Tue, 10 Jan 2017 12:55:14 +0000 (+0100) Subject: patch 8.0.0165: ubsan warns for integer overflow X-Git-Tag: v8.0.0165 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f446b48ff0bffae2b453cd4f9e3c25dfe363d29d;p=vim patch 8.0.0165: ubsan warns for integer overflow Problem: Ubsan warns for integer overflow. Solution: Swap two conditions. (Dominique Pelle) --- diff --git a/src/regexp_nfa.c b/src/regexp_nfa.c index 919dadcb0..de22dcbac 100644 --- a/src/regexp_nfa.c +++ b/src/regexp_nfa.c @@ -50,7 +50,7 @@ enum NFA_CONCAT, /* concatenate two previous items (postfix * only) */ NFA_OR, /* \| (postfix only) */ - NFA_STAR, /* greedy * (posfix only) */ + NFA_STAR, /* greedy * (postfix only) */ NFA_STAR_NONGREEDY, /* non-greedy * (postfix only) */ NFA_QUEST, /* greedy \? (postfix only) */ NFA_QUEST_NONGREEDY, /* non-greedy \? (postfix only) */ @@ -2169,7 +2169,7 @@ nfa_regpiece(void) * 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)) + && (maxval > 500 || maxval > minval + 200)) return FAIL; /* Ignore previous call to nfa_regatom() */ diff --git a/src/version.c b/src/version.c index 36143206a..b5039a01d 100644 --- a/src/version.c +++ b/src/version.c @@ -764,6 +764,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 165, /**/ 164, /**/