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

Include <endianness.h> also in base64 #985

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

th-otto
Copy link
Contributor

@th-otto th-otto commented Mar 24, 2024

Without this, base64 decoder (used to encode/decrypt the PKeys) does not work, resulting in division by zero

Without this, base64 decoder (used to encode/decrypt the PKeys)
does not work, resulting in division by zero
Copy link
Contributor

@OmniBlade OmniBlade left a comment

Choose a reason for hiding this comment

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

This PR touches mp.cpp as well as base64.cpp which is not mentioned in the PR description, is it intentional?

@th-otto
Copy link
Contributor Author

th-otto commented Mar 25, 2024

Maybe the PR description could be better, but the fixes in mp.cpp are needed as well.

@OmniBlade OmniBlade merged commit 4ed8f34 into TheAssemblyArmada:vanilla Mar 25, 2024
10 checks passed
@th-otto th-otto deleted the PR-38 branch March 25, 2024 10:06
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.

2 participants