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

[v9] Fixes potential cgo.Handle panic #13590

Merged
merged 7 commits into from
Jun 21, 2022

Conversation

ibeckermayer
Copy link
Contributor

Backport #13479 to branch/v9

@ibeckermayer ibeckermayer changed the base branch from master to branch/v9 June 16, 2022 21:00
@ibeckermayer
Copy link
Contributor Author

Note to self: Once this is merged, cherry-pick its merge commit to the top of the isaiah/backports-branch/v9 branch, and change

if err := cgoError(C.close_rdp(c.rustClient)); err != nil {

to

if err := C.close_rdp(c.rustClient); err != C.ErrCodeSuccess {

to backport #11616, #11714, and #12499

@ibeckermayer ibeckermayer enabled auto-merge (squash) June 17, 2022 20:16
@ibeckermayer ibeckermayer merged commit 70cb4d7 into branch/v9 Jun 21, 2022
@zmb3 zmb3 deleted the isaiah/backport-13479-branch/v9 branch April 26, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants