0
0
mirror of https://github.com/vim/vim.git synced 2025-09-25 03:54:15 -04:00

patch 8.0.0705: crash when there is an error in a timer callback

Problem:    Crash when there is an error in a timer callback. (Aron Griffis,
            Ozaki Kiichi)
Solution:   Check did_throw before discarding an exception.  NULLify
            current_exception when no longer valid.
This commit is contained in:
Bram Moolenaar
2017-07-10 22:12:10 +02:00
parent 163095f088
commit cae24be4a8
3 changed files with 11 additions and 3 deletions

View File

@@ -1235,7 +1235,7 @@ check_due_timer(void)
if (called_emsg) if (called_emsg)
{ {
++timer->tr_emsg_count; ++timer->tr_emsg_count;
if (!did_throw_save && current_exception != NULL) if (!did_throw_save && did_throw && current_exception != NULL)
discard_current_exception(); discard_current_exception();
} }
did_emsg = did_emsg_save; did_emsg = did_emsg_save;

View File

@@ -640,8 +640,11 @@ discard_exception(except_T *excp, int was_finished)
void void
discard_current_exception(void) discard_current_exception(void)
{ {
if (current_exception != NULL)
{
discard_exception(current_exception, FALSE); discard_exception(current_exception, FALSE);
current_exception = NULL; current_exception = NULL;
}
did_throw = FALSE; did_throw = FALSE;
need_rethrow = FALSE; need_rethrow = FALSE;
} }
@@ -1978,7 +1981,10 @@ enter_cleanup(cleanup_T *csp)
* there is an extra instance for every call of do_cmdline(), anyway. * there is an extra instance for every call of do_cmdline(), anyway.
*/ */
if (did_throw || need_rethrow) if (did_throw || need_rethrow)
{
csp->exception = current_exception; csp->exception = current_exception;
current_exception = NULL;
}
else else
{ {
csp->exception = NULL; csp->exception = NULL;

View File

@@ -764,6 +764,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
705,
/**/ /**/
704, 704,
/**/ /**/