diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2015-12-08 07:23:37 -0800 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2015-12-08 12:23:22 -0800 |
| commit | 9e0ff773ad5840af78ef0deeebb7da2f503eca32 (patch) | |
| tree | a8e6e7f86a160227754bb9d16db59b998560675f /src/test | |
| parent | 8864f2c83ac800881da34c3e835c931c081a8785 (diff) | |
| download | rust-9e0ff773ad5840af78ef0deeebb7da2f503eca32.tar.gz rust-9e0ff773ad5840af78ef0deeebb7da2f503eca32.zip | |
std: Use mem::replace in TLS initialization
Due to #30228 it's not currently sound to do `*ptr = Some(value)`, so instead use `mem::replace` which fixes the soundness hole for now.
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/run-pass/tls-init-on-init.rs | 51 |
1 files changed, 51 insertions, 0 deletions
diff --git a/src/test/run-pass/tls-init-on-init.rs b/src/test/run-pass/tls-init-on-init.rs new file mode 100644 index 00000000000..195b814492a --- /dev/null +++ b/src/test/run-pass/tls-init-on-init.rs @@ -0,0 +1,51 @@ +// Copyright 2015 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or +// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license +// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +#![feature(thread_local_state)] + +use std::thread::{self, LocalKeyState}; +use std::sync::atomic::{AtomicUsize, Ordering, ATOMIC_USIZE_INIT}; + +struct Foo { cnt: usize } + +thread_local!(static FOO: Foo = Foo::init()); + +static CNT: AtomicUsize = ATOMIC_USIZE_INIT; + +impl Foo { + fn init() -> Foo { + let cnt = CNT.fetch_add(1, Ordering::SeqCst); + if cnt == 0 { + FOO.with(|_| {}); + } + Foo { cnt: cnt } + } +} + +impl Drop for Foo { + fn drop(&mut self) { + if self.cnt == 1 { + FOO.with(|foo| assert_eq!(foo.cnt, 0)); + } else { + assert_eq!(self.cnt, 0); + match FOO.state() { + LocalKeyState::Valid => panic!("should not be in valid state"), + LocalKeyState::Uninitialized | + LocalKeyState::Destroyed => {} + } + } + } +} + +fn main() { + thread::spawn(|| { + FOO.with(|_| {}); + }).join().unwrap(); +} |
