From: Roland McGrath Date: Wed, 12 Mar 2014 22:42:13 +0000 (+0000) Subject: PR libstdc++/59392: Fix ARM EABI uncaught throw from unexpected exception handler X-Git-Tag: upstream/12.2.0~64293 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=90021b558c4d39ae9444b90f08f86c0aa2360e85;p=platform%2Fupstream%2Fgcc.git PR libstdc++/59392: Fix ARM EABI uncaught throw from unexpected exception handler libstdc++-v3/ PR libstdc++/59392 * libsupc++/eh_call.cc (__cxa_call_unexpected): Call __do_catch with the address of a null pointer, not with a null pointer to pointer. Copy comment for this case from eh_personality.cc:__cxa_call_unexpected. * testsuite/18_support/bad_exception/59392.cc: New file. Co-Authored-By: Mark Seaborn From-SVN: r208519 --- diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 7c92019..720ef1d 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,12 @@ +2014-03-12 Roland McGrath + Mark Seaborn + + PR libstdc++/59392 + * libsupc++/eh_call.cc (__cxa_call_unexpected): Call __do_catch with + the address of a null pointer, not with a null pointer to pointer. + Copy comment for this case from eh_personality.cc:__cxa_call_unexpected. + * testsuite/18_support/bad_exception/59392.cc: New file. + 2014-03-11 Jonathan Wakely PR libstdc++/60499 diff --git a/libstdc++-v3/libsupc++/eh_call.cc b/libstdc++-v3/libsupc++/eh_call.cc index dc32dd3..7677692 100644 --- a/libstdc++-v3/libsupc++/eh_call.cc +++ b/libstdc++-v3/libsupc++/eh_call.cc @@ -104,14 +104,14 @@ __cxa_call_unexpected(void* exc_obj_in) } end_catch_protect_obj; - __try - { + __try + { if (foreign_exception) std::unexpected(); else __unexpected(unexpectedHandler); } - __catch(...) + __catch(...) { /* See if the new exception matches the rtti list. */ if (foreign_exception) @@ -140,15 +140,19 @@ __cxa_call_unexpected(void* exc_obj_in) &new_ptr) != ctm_failed) __throw_exception_again; - if (catch_type->__do_catch(&bad_exc, 0, 1)) + // If the exception spec allows std::bad_exception, throw that. + // We don't have a thrown object to compare against, but since + // bad_exception doesn't have virtual bases, that's OK; just pass NULL. + void* obj = NULL; + if (catch_type->__do_catch(&bad_exc, &obj, 1)) bad_exception_allowed = true; } // If the exception spec allows std::bad_exception, throw that. -#ifdef __EXCEPTIONS +#ifdef __EXCEPTIONS if (bad_exception_allowed) throw std::bad_exception(); -#endif +#endif // Otherwise, die. __terminate(terminateHandler); diff --git a/libstdc++-v3/testsuite/18_support/bad_exception/59392.cc b/libstdc++-v3/testsuite/18_support/bad_exception/59392.cc new file mode 100644 index 0000000..c9b65a2 --- /dev/null +++ b/libstdc++-v3/testsuite/18_support/bad_exception/59392.cc @@ -0,0 +1,51 @@ +// Copyright (C) 2014 Free Software Foundation, Inc. +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 3, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING3. If not see +// . + +#include +#include + +class expected {}; +class unexpected {}; +class from_handler {}; + +static void func_with_exception_spec() throw(expected) +{ + throw unexpected(); +} + +static void unexpected_handler() +{ + throw from_handler(); +} + +static void terminate_handler() +{ + exit(0); +} + +// libstdc++/59392 +int main() +{ + std::set_unexpected(unexpected_handler); + std::set_terminate(terminate_handler); + try { + func_with_exception_spec(); + } catch (expected&) { + abort(); + } + abort(); +}