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

Non-constant time base64 #7

Open
Sc00bz opened this issue Aug 30, 2020 · 4 comments
Open

Non-constant time base64 #7

Sc00bz opened this issue Aug 30, 2020 · 4 comments

Comments

@Sc00bz
Copy link
Contributor

Sc00bz commented Aug 30, 2020

You should be using a constant time base64 encode/decode. I'd suggest using: https://github.com/Sc00bz/ConstTimeEncoding

@pro465
Copy link

pro465 commented Feb 22, 2023

what does that mean? skimming over the c++ src, it seems to be O(N) time...

@jedisct1
Copy link

what does that mean? skimming over the c++ src, it seems to be O(N) time...

Table lookups are not guaranteed to be free of side channels.

@pro465
Copy link

pro465 commented Feb 22, 2023

oh so that's what you meant. thanks for the speedy response @jedisct1, and yeah i guess then @Sc00bz is right about saying that the author "should be using" that, even though it sounded like they are advertising their library.

@henrycg
Copy link
Owner

henrycg commented May 29, 2023

Please feel free to open a PR for this and I can review and merge it.

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

No branches or pull requests

4 participants