Skip to content

Commit

Permalink
patch 8.2.3486: illegal memory access with invalid sequence of commands
Browse files Browse the repository at this point in the history
Problem:    Illegal memory access with invalid sequence of commands.
Solution:   Do not call leave_block() when not in a try block. (closes #8966)
            Reset did_emsg so that exception is shown as an error.
  • Loading branch information
brammool committed Oct 6, 2021
1 parent edd7a47 commit cce81e9
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 2 deletions.
10 changes: 8 additions & 2 deletions src/ex_eval.c
Expand Up @@ -1385,7 +1385,7 @@ ex_endwhile(exarg_T *eap)
eap->errmsg = _(err);
else
{
fl = cstack->cs_flags[cstack->cs_idx];
fl = cstack->cs_flags[cstack->cs_idx];
if (!(fl & csf))
{
// If we are in a ":while" or ":for" but used the wrong endloop
Expand Down Expand Up @@ -2007,6 +2007,7 @@ ex_endtry(exarg_T *eap)
if (!(cstack->cs_flags[cstack->cs_idx] & CSF_TRY))
{
eap->errmsg = get_end_emsg(cstack);

// Find the matching ":try" and report what's missing.
idx = cstack->cs_idx;
do
Expand All @@ -2025,6 +2026,9 @@ ex_endtry(exarg_T *eap)
*/
if (did_throw)
discard_current_exception();

// report eap->errmsg, also when there already was an error
did_emsg = FALSE;
}
else
{
Expand Down Expand Up @@ -2105,7 +2109,9 @@ ex_endtry(exarg_T *eap)
*/
(void)cleanup_conditionals(cstack, CSF_TRY | CSF_SILENT, TRUE);

leave_block(cstack);
if (cstack->cs_idx >= 0
&& (cstack->cs_flags[cstack->cs_idx] & CSF_TRY))
leave_block(cstack);
--cstack->cs_trylevel;

if (!skip)
Expand Down
20 changes: 20 additions & 0 deletions src/testdir/test_trycatch.vim
Expand Up @@ -2295,5 +2295,25 @@ func Test_error_in_catch_and_finally()
call delete('XtestCatchAndFinally')
endfunc

" This was causing an illegal memory access
func Test_leave_block_in_endtry_not_called()
let lines =<< trim END
vim9script
try #
for x in []
if
endwhile
if
endtry
END
call writefile(lines, 'XtestEndtry')
try
source XtestEndtry
catch /E171:/
endtry

call delete('XtestEndtry')
endfunc

" Modeline {{{1
" vim: ts=8 sw=2 sts=2 expandtab tw=80 fdm=marker
2 changes: 2 additions & 0 deletions src/version.c
Expand Up @@ -757,6 +757,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
3486,
/**/
3485,
/**/
Expand Down

0 comments on commit cce81e9

Please sign in to comment.