Re: Segfault with surround.vim on long lines (450 chars), via matchstr()

[Christian Brabandt]

Am 2014-11-20 00:40, schrieb Daniel Hahler:
line 41: let append = matchstr(keeper,'\_s\@<!\s*$')

That is enough to trigger the bug, with a long enough line.

I believe there was still a problem in regexp_nfa.c which
prevented patch 497 to work properly and causes the
"conditional jump depends on uninitialised value"
in valgrind.

Can you check, if the attached patch fixes it for you?

Best,
Christian

--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
diff --git a/src/regexp_nfa.c b/src/regexp_nfa.c
--- a/src/regexp_nfa.c
+++ b/src/regexp_nfa.c
@@ -7158,7 +7158,7 @@ nfa_regexec_nl(rmp, line, col, line_lbr)
     ireg_icombine = FALSE;
 #endif
     ireg_maxcol = 0;
-    return (nfa_regexec_both(line, col) != 0);
+    return nfa_regexec_both(line, col);
 }