Fix unsafe operation of wasm32::memory_atomic_notify

rust-lang · Oct 24, 2020 · d147f78 · d147f78
1 parent de87ae7
commit d147f78
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/library/std/src/sys/wasm/condvar_atomics.rs b/library/std/src/sys/wasm/condvar_atomics.rs
@@ -52,8 +52,9 @@ impl Condvar {
 
     #[inline]
     pub unsafe fn notify_all(&self) {
+        self.cnt.fetch_add(1, SeqCst);
+        // SAFETY: memory_atomic_notify()is always valid
         unsafe {
-            self.cnt.fetch_add(1, SeqCst);
             wasm32::memory_atomic_notify(self.ptr(), u32::MAX); // -1 == "wake everyone"
         }
     }