IMPORT: slz: fix the adler32 accumulators signedness on 32-bit

slz_adler32_block() unfortunately uses a signed long as the crc accumulator
instead of an unsigned one, meaning that for CRC values where the 32th bit
is set on 32-bit machines, the right shift will drag sign bits and corrupt
it. This only affects zlib streams on 32-bit systems (rfc1950) and has
been there for a very long time, showing that the zlib format is really
not much used in target environments.

The fix is trivial, just change the accumulators to unsigned long.

This is libslz upstream commit 912a707525fd2d6a63c9884ef02f69e7379c304c

diff --git a/src/slz_common.c b/src/slz_common.c
index d117688b7dddb837a4f0608930e5b470169b5851..62ff4f747be8dc630656a38f6f75a3195d5b5b15 100644 (file)
--- a/src/slz_common.c
+++ b/src/slz_common.c
@@ -200,8 +200,8 @@ uint32_t slz_adler32_by1(uint32_t crc, const unsigned char *buf, int len)
  */
 uint32_t slz_adler32_block(uint32_t crc, const unsigned char *buf, long len)
 {
-       long s1 = crc & 0xffff;
-       long s2 = (crc >> 16);
+       unsigned long s1 = crc & 0xffff;
+       unsigned long s2 = (crc >> 16);
        long blk;
        long n;