Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Shutdown requests might free their memory before running the callback #63

Closed
Tragetaschen opened this issue Feb 11, 2015 · 1 comment
Closed

Shutdown requests might free their memory before running the callback #63

Tragetaschen opened this issue Feb 11, 2015 · 1 comment

Comments

@Tragetaschen
Copy link
Contributor

UvShutdownReq's base class UvMemory only allocates a weak GC-handle. This means between creating the class in a Connection and having libuv running the callback method, it's free for collection and the Finalizer might run, which frees the memory
Tragetaschen pushed a commit to Tragetaschen/KestrelHttpServer that referenced this issue Feb 11, 2015
Moved the self keepalive into the base class so the shutdown request …
b62131c 
…has it too.

Fixes aspnet#63
@Tragetaschen
Copy link
Contributor Author

#60 includes a fix

Tragetaschen pushed a commit to Tragetaschen/KestrelHttpServer that referenced this issue Mar 14, 2015
Moved the self keepalive into the base class so the shutdown request …
0133041 
…has it too.

Fixes aspnet#63
Tragetaschen pushed a commit to Tragetaschen/KestrelHttpServer that referenced this issue Mar 14, 2015
Moved the self keepalive into the base class so the shutdown request …
6ff30ce 
…has it too.

Fixes aspnet#63
Tragetaschen pushed a commit to Tragetaschen/KestrelHttpServer that referenced this issue Apr 30, 2015
Moved the self keepalive into the base class so the shutdown request …
797c279 
…has it too.

Fixes aspnet#63
This was referenced Jun 24, 2015
Closed
Closed
@vinh84 vinh84 mentioned this issue Aug 27, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Tragetaschen