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

Windows issue Release vs Debug #17

Open
davens opened this issue Oct 4, 2018 · 4 comments
Open

Windows issue Release vs Debug #17

davens opened this issue Oct 4, 2018 · 4 comments

Comments

@davens
Copy link

davens commented Oct 4, 2018

Lovely library and been working without issues for osx. However, trying to use with windows and having issues. Debug works fine and the same output is generated (as on osx)

However Release comes out with something wrong. I don't have a windows box in front of me so did a remote debugging session over skype which is a fairly slow process and found that the issue begins after calling expandKey

https://github.com/bricke/Qt-AES/blob/master/qaesencryption.cpp#L397

The expanded key is different.

Please note there are also some warnings, for type mismatch in windows for example:
ret.lastIndexOf(0x80)

This could maybe be a clue to some of the issues.

@davens
Copy link
Author

davens commented Oct 4, 2018

Compiler msvc2017

@bricke
Copy link
Owner

bricke commented Oct 18, 2018

Thank you,

I just recently got a windows machine and I will try to debug this, if you find a solution and send me a PR I would gladly review it.

@rahbari
Copy link

rahbari commented Jun 16, 2019

I got wrong result with release build too although debug is ok, msvc 2017 x32.

@rahbari
Copy link

rahbari commented Jun 16, 2019

I updated to MSVS 2019 and it works now.

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

3 participants