Commit Graph

10 Commits

Author SHA1 Message Date
Miss Islington (bot) a512493371 bpo-31933: fix blake2 multi-byte params on big endian platforms (GH-4250) (#4262)
All Blake2 params have to be encoded in little-endian byte order. For
the two multi-byte integer params, leaf_length and node_offset, that
means that assigning a native-endian integer to them appears to work on
little-endian platforms, but gives the wrong result on big-endian. The
current libb2 API doesn't make that very clear, and @sneves is working
on new API functions in the GH issue above. In the meantime, we can work
around the problem by explicitly assigning little-endian values to the
parameter block.

See https://github.com/BLAKE2/libb2/issues/12.
(cherry picked from commit dcfb0e3c04)
2017-11-03 20:55:54 +01:00
Miss Islington (bot) f44a6299e2 Remove nested comments in blake2 (GH-4173) (#4214)
Replace occurence of nested comments in blake2 reference implementation
with preprocessor directive for disabling unused code.

`blake2s-load-xop.h` is conditionally pulled in only on chips with XOP
support, among others the AMD Bulldozer. The malformed comments in the
source file breaks the build of `hashlib`'s `_blake2` on GCC 6.3.0.

Official reference code on github uses `GH-if` so this change should be
uncontroversial.
(cherry picked from commit 388cd85e51)
2017-11-01 12:58:40 +00:00
Benjamin Peterson 47ff0734b8 more PY_LONG_LONG to long long 2016-09-08 09:15:54 -07:00
Christian Heimes 5940c535b0 Issue #26798: Coverity complains about potential memcpy() of overlapped regions. It doesn't hurt to use memmove() here. CID 1372514 / CID 1372515. Upstream https://github.com/BLAKE2/BLAKE2/issues/32 2016-09-08 13:40:25 +02:00
Christian Heimes dfb9ef1357 blake2: silence two more warnings on platforms with size_t < uint64_t. Don't use SSE2 when cross-compiling 2016-09-07 11:39:21 +02:00
Christian Heimes cc554b65f1 Silence two warnings in blake2. key_length is between 0 and 64 (block size). 2016-09-07 02:49:11 +02:00
Christian Heimes dc5a3fe4ef Issue #26798: for loop initial declarations, take 3 2016-09-07 00:32:06 +02:00
Christian Heimes 9c2f304103 Issue #26798: for loop initial declarations, take 2 2016-09-07 00:09:22 +02:00
Christian Heimes 87bf0febcb Issue #26798: for loop initial declarations are only allowed in C99 or C11 mode 2016-09-06 23:18:03 +02:00
Christian Heimes 121b9487d1 Issue #26798: Add BLAKE2 (blake2b and blake2s) to hashlib. 2016-09-06 22:03:25 +02:00