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.0871: using freed memory when clearing augroup at more prompt

Browse Source 
Problem:    Using freed memory when clearing augroup at more prompt.
Solution:   Delay clearing augroup until it's safe. (closes #11441)
This commit is contained in:
Bram Moolenaar
2022-11-13 17:53:46 +00:00
parent 2f7bfe66a1
commit 3b014befa0
3 changed files with 41 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/autocmd.c
View File

@@ -296,9 +296,14 @@ show_autocmd(AutoPat *ap, event_T event)
if (ap->pat == NULL) // pattern has been removed
return;
// Make sure no info referenced by "ap" is cleared, e.g. when a timer
// clears an augroup. Jump to "theend" after this!
// "ap->pat" may be cleared anyway.
++autocmd_busy;
msg_putchar('\n');
if (got_int)
return;
goto theend;
if (event != last_event || ap->group != last_group)
{
if (ap->group != AUGROUP_DEFAULT)
@@ -314,8 +319,12 @@ show_autocmd(AutoPat *ap, event_T event)
last_group = ap->group;
msg_putchar('\n');
if (got_int)
return;
goto theend;
}
if (ap->pat == NULL)
goto theend; // timer might have cleared the pattern or group
msg_col = 4;
msg_outtrans(ap->pat);
@@ -328,21 +337,24 @@ show_autocmd(AutoPat *ap, event_T event)
msg_putchar('\n');
msg_col = 14;
if (got_int)
return;
goto theend;
msg_outtrans(ac->cmd);
#ifdef FEAT_EVAL
if (p_verbose > 0)
last_set_msg(ac->script_ctx);
#endif
if (got_int)
return;
goto theend;
if (ac->next != NULL)
{
msg_putchar('\n');
if (got_int)
return;
goto theend;
}
}
theend:
--autocmd_busy;
}
/*