aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-09-29 04:34:16 -04:00
Code Activity

patch 9.0.0530: using freed memory when autocmd changes mark

Browse Source 
Problem:    Using freed memory when autocmd changes mark.
Solution:   Copy the mark before editing another buffer.
This commit is contained in:
Bram Moolenaar
2022-09-21 13:07:22 +01:00
parent 8eaee1fa5c
commit 8ecfa2c56b
3 changed files with 22 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/mark.c
View File

@@ -221,17 +221,19 @@ movemark(int count)
fname2fnum(jmp);
if (jmp->fmark.fnum != curbuf->b_fnum)
{
// jump to other file
if (buflist_findnr(jmp->fmark.fnum) == NULL)
// Make a copy, an autocommand may make "jmp" invalid.
fmark_T fmark = jmp->fmark;
// jump to the file with the mark
if (buflist_findnr(fmark.fnum) == NULL)
{ // Skip this one ..
count += count < 0 ? -1 : 1;
continue;
}
if (buflist_getfile(jmp->fmark.fnum, jmp->fmark.mark.lnum,
0, FALSE) == FAIL)
if (buflist_getfile(fmark.fnum, fmark.mark.lnum, 0, FALSE) == FAIL)
return (pos_T *)NULL;
// Set lnum again, autocommands my have changed it
curwin->w_cursor = jmp->fmark.mark;
curwin->w_cursor = fmark.mark;
pos = (pos_T *)-1;
}
else