Fix ASAN error and enable ASAN in CI

[allada]

Enables ASAN tests in CI and fixes the circular reference it found.

---

This change is 

[allada]

+@aaronmondal

Reviewable status: 0 of 5 files reviewed, all discussions resolved (waiting on @aaronmondal)

[aaronmondal]

Interesting. Seems like i went into the wrong direction when trying to fix this 😆

I have a few small changes to tsan ready which seem to distill tsan issues down to 3 tests.

Addresses the Asan issues in #188

Reviewed 5 of 5 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @allada)

[allada]

Yeah any time you see unsafe your eyes immediately go "oh, that's the issue", but in reality any time unsafe is actually used it is usually really carefully thought about to make sure it really is safe.

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @allada)