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

crypto/tls: panic in TransportTLS handshake in Windows crashes app with panic [1.9 backport] #25034

Closed
gopherbot opened this issue Apr 24, 2018 · 3 comments
Milestone

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Apr 24, 2018

@FiloSottile requested issue #21376 to be considered for backport to the next 1.9 minor release.

@gopherbot please file this for backport.

See @aclements's comment #21376 (comment) about having to develop a different fix from CL 106275 in order not to break a public API.

@gopherbot

This comment has been minimized.

Copy link
Author

@gopherbot gopherbot commented May 8, 2018

Change https://golang.org/cl/112095 mentions this issue: [release-branch.go1.9] crypto/x509: copy and use adjusted syscall.CertChainPolicyPara

@gopherbot

This comment has been minimized.

Copy link
Author

@gopherbot gopherbot commented May 9, 2018

Change https://golang.org/cl/112179 mentions this issue: [release-branch.go1.9] crypto/x509: copy and use adjusted syscall.CertChainPolicyPara

@gopherbot

This comment has been minimized.

Copy link
Author

@gopherbot gopherbot commented May 9, 2018

Closed by merging f31a132 to release-branch.go1.9.

@gopherbot gopherbot closed this May 9, 2018
gopherbot pushed a commit that referenced this issue May 9, 2018
…tChainPolicyPara

As discussed in issue #21376, it is unsafe to have
syscall.CertChainPolicyPara.ExtraPolicyPara uintptr -
it has to be a pointer type. So copy syscall.CertChainPolicyPara
into crypto/tls package, make ExtraPolicyPara unsafe.Pointer,
and use new struct instead of syscall.CertChainPolicyPara.

Fixes #25034

Change-Id: If914af056cbbb0c4d93ffaa915b3d2cb5ecad0cd
Reviewed-on: https://go-review.googlesource.com/111715
Reviewed-by: Austin Clements <austin@google.com>
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-on: https://go-review.googlesource.com/112179
Reviewed-by: Filippo Valsorda <filippo@golang.org>
Run-TryBot: Filippo Valsorda <filippo@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
@golang golang locked and limited conversation to collaborators May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.