-
Notifications
You must be signed in to change notification settings - Fork 6
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
[BUG] bad $length #15
Comments
phpseclib2_compat/src/Crypt/Base.php Lines 231 to 234 in 3468997
I tried to workaround first but this hardcoded namespace complicates things, please kindly accept the PR #16 thanks |
This has been merged. That said, do understand that this is mainly a one man project, this isn't my full time job and I have other responsibilities above and beyond phpseclib so please keep that in mind when awaiting responses. Anyway, thanks for the bug report / pull request! |
Thanks! Yes, I'm aware there are no obligations, but as I could not find another work around, I decided to send the 2nd message. Appreciate the quick response! |
https://github.com/phpseclib/phpseclib/blob/44d5ca2ab572b047caf09bf7e2397a8fb6bf7b10/phpseclib/Crypt/Rijndael.php#L175-L180
phpseclib2_compat/src/Crypt/Rijndael.php
Lines 75 to 85 in 3468997
$length
is shifted 5 bits by this repo, not what the actual phpseclib3 expectsThe text was updated successfully, but these errors were encountered: