Merge pull request #55 from tmiasko/current-thread-waker

yoshuawuyts · web-flow · commit 954363af876f · 2020-02-17T13:17:46.000+01:00
Fix off-by-one in current thread waker refcount
diff --git a/src/native/global.rs b/src/native/global.rs
@@ -102,5 +102,5 @@ fn raw_drop(ptr: *const ()) {
 
 fn current_thread_waker() -> Waker {
     let thread = Arc::new(thread::current());
-    unsafe { Waker::from_raw(raw_clone(Arc::into_raw(thread) as *const ())) }
+    unsafe { Waker::from_raw(RawWaker::new(Arc::into_raw(thread) as *const (), &VTABLE)) }
 }

Original file line number	Diff line number	Diff line change
`@@ -102,5 +102,5 @@ fn raw_drop(ptr: *const ()) {`
`102`	`102`
`103`	`103`	`fn current_thread_waker() -> Waker {`
`104`	`104`	`let thread = Arc::new(thread::current());`
`105`		`- unsafe { Waker::from_raw(raw_clone(Arc::into_raw(thread) as *const ())) }`
	`105`	`+ unsafe { Waker::from_raw(RawWaker::new(Arc::into_raw(thread) as *const (), &VTABLE)) }`
`106`	`106`	`}`