Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stream thread synchronization problem #151

Open
2 tasks
zippy opened this issue May 26, 2016 · 0 comments
Open
2 tasks

Stream thread synchronization problem #151

zippy opened this issue May 26, 2016 · 0 comments
Assignees
Milestone

Comments

@zippy
Copy link
Owner

zippy commented May 26, 2016

The current code we use for HTTP edge receptor reveals a synchronization problem:

The loop ends with a call to CLOSE_STREAM, which can end up shutting down the stream before the response from the other thread gets sent. This shows up with the system throwing a "bad stream type" error because the stream gets deallocated and pointer to bogus memory.

This means we have to do two things:

  • add some way to do reference counting on stream objects so that we don't get reads on deallocated memory
  • re-write the HTTP edge receptor code to synchronize properly on only close the stream when everything is really done.
@zippy zippy self-assigned this May 26, 2016
@zippy zippy added this to the May milestone May 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant