Yegappan Lakshmanan wrote: > On Sat, Apr 30, 2016 at 10:50 AM, Bram Moolenaar <[email protected]> wrote: > > > > Anton Lindqvist wrote: > > > >> > > I managed to crash Vim yesterday due to a off-by-one allocation bug > >> > > related to the quickfix changes. Fixed in the attached patch. > >> > > >> > Well, if I run test_quickfix after including this patch, Vim crashes. > >> > I found that a statement was missing, setting linelen to len. > >> > I did some minor cleanups. > >> > >> Nice catch! > >> > >> > I'll include it now. However, the code to increase the size of the > >> > buffer is repeated three times, would be nice to put it in one place. > >> > > Vim crashes when running the quickfix tests on a OS X system with the > latest Vim (1811). The fix for this crash is attached.
Thanks. I didn't see the crash but running test_quickfix under valgrind does show the problem. -- The Law, in its majestic equality, forbids the rich, as well as the poor, to sleep under the bridges, to beg in the streets, and to steal bread. -- Anatole France /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org /// -- -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
