[CVE-2016-9843] Avoid pre-decrement of pointer in big-endian CRC calculation. 34/253434/1
authorMark Adler <madler@alumni.caltech.edu>
Thu, 29 Sep 2016 03:20:25 +0000 (20:20 -0700)
committerDongHun Kwak <dh0128.kwak@samsung.com>
Wed, 10 Feb 2021 04:58:11 +0000 (13:58 +0900)
commite2f48de7d886abde5d7572ae09117aa49f319792
treedca2a9b7d0e38dd03a2b45831a2e462d4e6fad8d
parentdd0ea6ced9cdb09a6e17c1216ac650c3440e38e2
[CVE-2016-9843] 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.

Change-Id: I3af497cf1229a09cdc2f2c20934b29add514084a
Signed-off-by: DongHun Kwak <dh0128.kwak@samsung.com>
zlib/crc32.c