Skip to content

Commit 9f44634

Browse files
committed
Fix for issue rust-lang#4547
1 parent 9d37d03 commit 9f44634

File tree

2 files changed

+21
-1
lines changed

2 files changed

+21
-1
lines changed

src/rt/rust_task.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,9 @@ cleanup_task(cleanup_args *args) {
129129
// assert(task->task_local_data != NULL);
130130
task->task_local_data_cleanup(task->task_local_data);
131131
task->task_local_data = NULL;
132-
} else if (threw_exception && task->id == INIT_TASK_ID) {
132+
}
133+
134+
if (threw_exception && task->id == INIT_TASK_ID) {
133135
// Edge case: If main never spawns any tasks, but fails anyway, TLS
134136
// won't be around to take down the kernel (task.rs:kill_taskgroup,
135137
// rust_task_kill_all). Do it here instead.

src/test/run-fail/issue-4547.rs

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
// Copyright 2013 The Rust Project Developers. See the COPYRIGHT
2+
// file at the top-level directory of this distribution and at
3+
// http://rust-lang.org/COPYRIGHT.
4+
//
5+
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6+
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7+
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8+
// option. This file may not be copied, modified, or distributed
9+
// except according to those terms.
10+
11+
//error-pattern:explicit failure
12+
13+
// Regression test for issue 4547
14+
15+
pub fn main() {
16+
os::args();
17+
fail!();
18+
}

0 commit comments

Comments
 (0)