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

cmd/cgo: detect go pointers #8839

Closed
rsc opened this issue Sep 29, 2014 · 6 comments
Closed

cmd/cgo: detect go pointers #8839

rsc opened this issue Sep 29, 2014 · 6 comments

Comments

@rsc
Copy link
Contributor

rsc commented Sep 29, 2014

detect go pointers being passed indirectly to c and stop them.
@griesemer
Copy link
Contributor

Comment 1:

Labels changed: added repo-main.

@frederich
Copy link
Contributor

Comment 2:

Russ, can you please provide more detailed information? I'd like to fix it, but don't
know where to start.
Jens

@rsc
Copy link
Contributor Author

rsc commented Oct 17, 2014

Comment 3:

I appreciate the offer but I'd rather take care of this myself. I have it partly done.

@frederich
Copy link
Contributor

Comment 4:

Ok

@rsc
Copy link
Contributor Author

rsc commented Oct 28, 2014

Comment 5:

We have decided not to do this for Go 1.4.
The plans for future releases are still up in the air.

Labels changed: added release-go1.5, removed release-go1.4.

@rsc rsc self-assigned this Oct 28, 2014
@bradfitz bradfitz modified the milestone: Go1.5 Dec 16, 2014
@rsc rsc removed accepted labels Apr 14, 2015
@rsc rsc modified the milestones: Unplanned, Go1.5 Jun 8, 2015
@ALTree
Copy link
Member

ALTree commented Jan 19, 2017

I believe this was fixed in go1.6, which shipped with the cgo runtime checks
https://golang.org/doc/go1.6#cgo

@ALTree ALTree closed this as completed Jan 19, 2017
@golang golang locked and limited conversation to collaborators Jan 19, 2018
@rsc rsc removed their assignment Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants