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

Fix handling for ALG_ID_ECDH_P384 #181

Conversation

Thulium-Drake
Copy link

@Thulium-Drake Thulium-Drake commented Mar 4, 2023

This fixes a missing case statement for handling ALG_ID_ECDH_P384.

This has at least fixed Anno 1800 multiplayer session negotiation, which did not work at all with Proton 7.0-6 (the one provided in Steam), but works like a charm after this patch has been applied to it.

The other PR (#173) adds extra code that is not required and already is present at line 1406 in wine/dlls/bcrypt/bcrypt_main.c

… fixes Anno 1800 multiplayer session negotiation
Copy link

@cachpachios cachpachios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Built the change and (finally) played Anno 1800 multiplayer successfully.
Such a small thing... yet...
Let's hope this gets merged ASAP.

Copy link

@PC-Heini PC-Heini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very important addition

Copy link

@Sulavius Sulavius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that the above change works.

@Etaash-mathamsetty
Copy link
Contributor

Etaash-mathamsetty commented Mar 16, 2023

this has been fixedin upstream wine already

@CvRXX
Copy link

CvRXX commented Mar 17, 2023

this has been fixedin upstream wine already

Do you have a commit hash for the fix?

@Etaash-mathamsetty
Copy link
Contributor

5253c8d

@Thulium-Drake
Copy link
Author

Closing as upstream fixed the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants