Dominique Pellé wrote:

> Bram Moolenaar <[email protected]> wrote:
> 
> > Patch 8.0.0224
> > Problem:    When 'fileformats' is changed in a BufReadPre auto command, it
> >             does not take effect in readfile(). (Gary Johnson)
> > Solution:   Check the value of 'fileformats' after executing auto commands.
> >             (Christian Brabandt)
> > Files:      src/fileio.c, src/testdir/test_fileformat.vim
> >
> >
> > *** ../vim-8.0.0223/src/fileio.c        2017-01-13 21:59:59.327172086
> 
> 
> Running tests with vim-8.0.225 built with asan, I see the
> following crash (head-buffer-overflow).  Since I see fileio.c
> in the stack, I assume that the bug is caused by patch 8.0.224:
> 
> =================================================================
> ==5073==ERROR: AddressSanitizer: heap-buffer-overflow on address
> 0x6310000a07ff at pc 0x000000a15d57 bp 0x7ffe2637ca50 sp
> 0x7ffe2637ca48
> READ of size 1 at 0x6310000a07ff thread T0
>     #0 0xa15d56 in readfile /home/pel/sb/vim/src/fileio.c:2254
>     #1 0x51c1b9 in open_buffer /home/pel/sb/vim/src/buffer.c:236

I suspect this is not caused by the patch, but because the test reads a
file in DOS mode with the first line lacking a CR.

Hmm, valgrind does not see this error.  And the address sanitizer does
not work: 
ERROR: AddressSanitizer failed to allocate 0xdfff0001000 (15392894357504) bytes 
at address 2008fff7000 (errno: 12)

I don't have that many bytes!  Didn't find a way to avoid that.

-- 
ARTHUR:    Well, it doesn't matter.  Will you go and tell your master that
           Arthur from the Court of Camelot is here.
GUARD #1:  Listen, in order to maintain air-speed velocity, a swallow
           needs to beat its wings 43 times every second, right?
ARTHUR:    Please!
                                  The Quest for the Holy Grail (Monty Python)

 /// 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.

Raspunde prin e-mail lui