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.

Sign up for GitHub

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

Jump to bottom

Double-free of ivecs when sending across channel #698

Closed
robarnold opened this issue Jul 15, 2011 · 5 comments
Closed

Double-free of ivecs when sending across channel #698

robarnold opened this issue Jul 15, 2011 · 5 comments
Assignees
@pcwalton

Comments

@robarnold
Copy link
Contributor

Finally found a minimal-ish testcase! This is blocking me from getting reading working for my async-io work

https://gist.github.com/1083866
@ghost ghost assigned pcwalton Jul 15, 2011
@eholk
Copy link
Contributor

eholk commented Jul 15, 2011

I bet this is the move semantics for channels issue again. Probably both ends drop the reference and then free it...

@robarnold
Copy link
Contributor Author

I should also note that if you drop the array length to 4 and the double-free problem goes away.

@robarnold
Copy link
Contributor Author

Not sure if you looked at it yet but I minimized the testcase completely now.

@brson
Copy link
Contributor

brson commented Jul 23, 2011

The test case works for me now.

@eholk
Copy link
Contributor

eholk commented Jul 23, 2011

This was probably fixed by 54be5b0.

@eholk eholk closed this as completed Jul 23, 2011
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
@bors
Auto merge of rust-lang#698 - ndusart:in_addr_t, r=alexcrichton
27a319e 
change type of INADDR constants to in_addr_t

These should be `in_addr_t` (u32) instead of `c_int`

This is a breaking change but it is just breaking newlib platforms for now.

@alexcrichton Do you prefer that I split the definition in submodules, to keep `c_int` in newlib and update the others ?
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this issue Mar 7, 2023
@tgymnich
Remove unused variable (rust-lang#698)
c16cec9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcwalton pcwalton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@eholk @brson @pcwalton @robarnold