Contribution description

Incorrect use of sizeof for string length in test test_hashes_sha512_hash_update_twice.

static const char *teststring is a pointer, therefore sizeof(teststring) returns the pointer size instead of the intended string length. This PR uses strlen instead of sizeof to fix this. Alternatively, we could change the type of the string to static const char teststring[].

Note the subtle difference between sizeof(char[]) and strlen: The former includes the null terminator, while strlen does not. This should not change the test behavior.

Edit:
This PR changes the type to static const char teststring[] so that sizeof returns the string length (including the null terminator).

Testing procedure

Compile and run unittests.

Note for 32-bit architectures: The test also passed before, but only used 4 and 3+1 bytes of the string.