foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-12-22 15:24:46 -05:00
Code Issues Packages 1 Wiki Activity

fix(core/runtime): always cancel termination in exception handling (#15514)

Browse source
This commit is contained in:
Nayeem Rahman 2022-08-21 12:57:10 +01:00 committed by GitHub
parent 3caec9721e
commit e39d4e3e7f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 22 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
cli/tests/integration/run_tests.rs
View file

@ -2738,6 +2738,13 @@ itest!(report_error_handled {
output: "report_error_handled.ts.out", output: "report_error_handled.ts.out",
}); });
// Regression test for https://github.com/denoland/deno/issues/15513.
itest!(report_error_end_of_program {
args: "run --quiet report_error_end_of_program.ts",
output: "report_error_end_of_program.ts.out",
exit_code: 1,
});
itest!(spawn_stdout_inherit { itest!(spawn_stdout_inherit {
args: "run --quiet --unstable -A spawn_stdout_inherit.ts", args: "run --quiet --unstable -A spawn_stdout_inherit.ts",
output: "spawn_stdout_inherit.ts.out", output: "spawn_stdout_inherit.ts.out",

1
cli/tests/testdata/report_error_end_of_program.ts vendored Normal file
View file

@ -0,0 +1 @@
reportError(new Error("foo"));

4
cli/tests/testdata/report_error_end_of_program.ts.out vendored Normal file
View file

@ -0,0 +1,4 @@
error: Uncaught Error: foo
reportError(new Error("foo"));
^
at [WILDCARD]/report_error_end_of_program.ts:1:13

18
core/runtime.rs
View file

@ -1156,14 +1156,14 @@ pub(crate) fn exception_to_err_result<'s, T>(
) -> Result<T, Error> { ) -> Result<T, Error> {
let state_rc = JsRuntime::state(scope); let state_rc = JsRuntime::state(scope);
let is_terminating_exception = scope.is_execution_terminating(); let was_terminating_execution = scope.is_execution_terminating();
let mut exception = exception; // If TerminateExecution was called, cancel isolate termination so that the
// exception can be created. Note that `scope.is_execution_terminating()` may
if is_terminating_exception { // have returned false if TerminateExecution was indeed called but there was
// TerminateExecution was called. Cancel isolate termination so that the // no JS to execute after the call.
// exception can be created..
scope.cancel_terminate_execution(); scope.cancel_terminate_execution();
let mut exception = exception;
{
// If the termination is the result of a `Deno.core.terminate` call, we want // If the termination is the result of a `Deno.core.terminate` call, we want
// to use the exception that was passed to it rather than the exception that // to use the exception that was passed to it rather than the exception that
// was passed to this function. // was passed to this function.
@ -1191,8 +1191,8 @@ pub(crate) fn exception_to_err_result<'s, T>(
); );
} }
if is_terminating_exception { if was_terminating_execution {
// Re-enable exception termination. // Resume exception termination.
scope.terminate_execution(); scope.terminate_execution();
} }