From 80c3bec9fefa7a32ca2613d9a9f84046818cec6a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomasz=20Mi=C4=85sko?= Date: Fri, 31 Jan 2020 00:00:00 +0000 Subject: [PATCH] Remove incorrect debug assertions from catch_unwind Previously the debug assertions in the implementation of catch_unwind used to verify consistency of the panic count by checking that the count is zero just before leaving the function. This incorrectly assumed that no panic was in progress when entering `catch_unwind`. --- src/libstd/panicking.rs | 2 -- .../issues/issue-68696-catch-during-unwind.rs | 26 +++++++++++++++++++ 2 files changed, 26 insertions(+), 2 deletions(-) create mode 100644 src/test/ui/issues/issue-68696-catch-during-unwind.rs diff --git a/src/libstd/panicking.rs b/src/libstd/panicking.rs index fee7ace92eaa3..54a57b4444c2f 100644 --- a/src/libstd/panicking.rs +++ b/src/libstd/panicking.rs @@ -286,11 +286,9 @@ pub unsafe fn r#try R>(f: F) -> Result> ); return if r == 0 { - debug_assert!(update_panic_count(0) == 0); Ok(ManuallyDrop::into_inner(data.r)) } else { update_panic_count(-1); - debug_assert!(update_panic_count(0) == 0); Err(mem::transmute(raw::TraitObject { data: any_data as *mut _, vtable: any_vtable as *mut _, diff --git a/src/test/ui/issues/issue-68696-catch-during-unwind.rs b/src/test/ui/issues/issue-68696-catch-during-unwind.rs new file mode 100644 index 0000000000000..d042bed225d17 --- /dev/null +++ b/src/test/ui/issues/issue-68696-catch-during-unwind.rs @@ -0,0 +1,26 @@ +// Checks that catch_unwind can be used if unwinding is already in progress. +// Used to fail when standard library had been compiled with debug assertions, +// due to incorrect assumption that a current thread is not panicking when +// entering the catch_unwind. +// +// run-pass +// ignore-wasm no panic support +// ignore-emscripten no panic support + +use std::panic::catch_unwind; + +#[derive(Default)] +struct Guard; + +impl Drop for Guard { + fn drop(&mut self) { + let _ = catch_unwind(|| {}); + } +} + +fn main() { + let _ = catch_unwind(|| { + let _guard = Guard::default(); + panic!(); + }); +}