]> git.lizzy.rs Git - rust.git/commitdiff
Disable at_exit handlers
authorAaron Turon <aturon@mozilla.com>
Fri, 19 Dec 2014 07:29:30 +0000 (23:29 -0800)
committerAaron Turon <aturon@mozilla.com>
Fri, 19 Dec 2014 07:35:53 +0000 (23:35 -0800)
The [final step](https://github.com/rust-lang/rust/pull/19654) of
runtime removal changes the threading/process model so that the process
shuts down when the main thread exits. But several shared resources,
like the helper thread for timeouts, are shut down when the main thread
exits (but before the process ends), and they are not prepared to be
used after shut down, but other threads may try to access them during
the shutdown sequence of the main thread.

As an interim solution, the `at_exit` cleanup routine is simply skipped.

Ultimately, these resources should be made to safely handle asynchronous
shutdown, usually by panicking if called from a detached thread when the
main thread is ending.

See issue for details https://github.com/rust-lang/rust/issues/20012

This is a [breaking-change] for anyone relying on `at_exit`.

src/libstd/rt/mod.rs

index fd50d845716b98be5617803e640ad045dfbb3fdc..8d9c1268e7e33dc51f09ceea84aa520b046ce37b 100644 (file)
@@ -162,5 +162,8 @@ pub fn at_exit<F:FnOnce()+Send>(f: F) {
 pub unsafe fn cleanup() {
     args::cleanup();
     sys::stack_overflow::cleanup();
-    at_exit_imp::cleanup();
+    // FIXME: (#20012): the resources being cleaned up by at_exit
+    // currently are not prepared for cleanup to happen asynchronously
+    // with detached threads using the resources; for now, we leak.
+    // at_exit_imp::cleanup();
 }