Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

possible use after "free" (move) due to broken unsafe use of generator resume #2

Closed
stbuehler opened this issue Feb 9, 2019 · 1 comment

Comments

@stbuehler
Copy link

As mentioned in aep/aep.github.io#4 you must not move a generator after calling resume.

This means you need to box the generator lambda before calling it the first time in osaka-macros/src/lib.rs:37.

@aep
Copy link
Owner

aep commented Feb 9, 2019

yeah, i was hoping to get away with it because osaka is single threaded, but that probably wasn't a smart idea

@aep aep closed this as completed in b19622c Apr 10, 2019
aep pushed a commit that referenced this issue Apr 10, 2019
aep added a commit that referenced this issue Apr 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants