]> git.lizzy.rs Git - zlib.git/commit
Avoid pre-decrement of pointer in big-endian CRC calculation.
authorMark Adler <madler@alumni.caltech.edu>
Thu, 29 Sep 2016 03:20:25 +0000 (20:20 -0700)
committerMark Adler <madler@alumni.caltech.edu>
Thu, 29 Sep 2016 03:48:38 +0000 (20:48 -0700)
commitd1d577490c15a0c6862473d7576352a9f18ef811
tree1d60d9150c67eb2c7fec06c5c7e1b61b6f7d9145
parent6a043145ca6e9c55184013841a67b2fef87e44c0
Avoid pre-decrement of pointer in big-endian CRC calculation.

There was a small optimization for PowerPCs to pre-increment a
pointer when accessing a word, instead of post-incrementing. This
required prefacing the loop with a decrement of the pointer,
possibly pointing before the object passed. This is not compliant
with the C standard, for which decrementing a pointer before its
allocated memory is undefined. When tested on a modern PowerPC
with a modern compiler, the optimization no longer has any effect.
Due to all that, and per the recommendation of a security audit of
the zlib code by Trail of Bits and TrustInSoft, in support of the
Mozilla Foundation, this "optimization" was removed, in order to
avoid the possibility of undefined behavior.
crc32.c