Uniqueness requirement for UnsafeCell aliases violated in oneshot.rs

[jganetsk]

async-heapless/src/oneshot.rs

Lines 175 to 177 in 835ae89

	/// NOTE(unsafe): This function must not be used concurrently with itself,
	/// but it can be used concurrently with put.
	pub unsafe fn take(&self) -> Option<T> {

It is claimed that put and take can be called concurrently. Both methods call UnsafeCell::get()

Ensure that the access is unique (no active references, mutable or not) when casting to &mut T

But put calls MaybeUninit::write() which casts it to &mut MaybeUninit<T>. You do not guarantee that the reference is unique, given that there could be a concurrent &MaybeUninit<T> in a concurrent call to take.

[jganetsk]

Nevermind, this is fine. The aliases should never exist at the same time.