memcpy produces differing code between GUI and CLI program if using __int128 extension

[kennbr34]

yaxa/crypt.c

Line 409 in cc5a22e

memcpy(&keyInt,keyBytes,sizeof(keyInt));

This call to memcpy causes keyInt to be reduced and produces an incorrect decryption as described in #8.

Replaced the call to memcpy with a loop

cryptint_t *keyIntPtr = &keyInt; for (uint8_t i = 0; i < sizeof(keyInt); i++) keyIntPtr[i] = keyBytes[i];

This solves the specific issue in #8 but since there are multiple calls to memcpy within the code it would be good to know specifically what is happening. Tested with memmove to make sure it wasn't overlapping memory, and that wasn't the issue ither.

[kennbr34]

cryptint_t *keyIntPtr = &keyInt; for (uint8_t i = 0; i < sizeof(keyInt); i++) keyIntPtr[i] = keyBytes[i];

Is a mistake. keyIntPtr should be of uint8_t. The previous code only works because it's overwriting the keyInt with 0s each time it's called.