bricke / Qt-AES

Native Qt AES encryption class
The Unlicense
501 stars 187 forks source link

Windows issue Release vs Debug #17

Open davens opened 5 years ago

davens commented 5 years ago

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 commented 5 years ago

Compiler msvc2017

bricke commented 5 years ago

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 commented 5 years ago

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

rahbari commented 5 years ago

I updated to MSVS 2019 and it works now.