Avoid recursive gzgetc() macro call.

Recursive macro calls are normally caught by the preprocessor and
avoided. This commit avoids the possibility of a problem entirely.

Make globals in examples local to compilation unit.

Add --warn option to ./configure, instead of environment variable.

Add inflateCodesUsed() function
Based on upstream commit 5370d99a2affe0b040550cffbc0ba8fa790594b3

Clean up type conversions.

Based on upstream 7096424f23df1b1813237fb5f8bc8f34cfcedd0c, but
modified heavily to match zlib-ng.

Note the violation of the strict aliasing rule in crc32.c.

See the comment for more details. This is in response to an issue
raised as a result of a security audit of the zlib code by Trail
of Bits and TrustInSoft, in support of the Mozilla Foundation.

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.

Remove offset pointer optimization in inftrees.c.

inftrees.c was subtracting an offset from a pointer to an array,
in order to provide a pointer that allowed indexing starting at
the offset. This is not compliant with the C standard, for which
the behavior of a pointer decremented before its allocated memory
is undefined. Per the recommendation of a security audit of the
zlib code by Trail of Bits and TrustInSoft, in support of the
Mozilla Foundation, this tiny optimization was removed, in order
to avoid the possibility of undefined behavior.

Use post-increment only in inffast.c.

An old inffast.c optimization turns out to not be optimal anymore
with modern compilers, and furthermore was not compliant with the
C standard, for which decrementing a pointer before its allocated
memory is undefined. 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.

Add option to not compute or check check values.

The undocumented (except in these commit comments) function
inflateValidate(strm, check) can be called after an inflateInit(),
inflateInit2(), or inflateReset2() with check equal to zero to
turn off the check value (CRC-32 or Adler-32) computation and
comparison. Calling with check not equal to zero turns checking
back on. This should only be called immediately after the init or
reset function. inflateReset() does not change the state, so a
previous inflateValidate() setting will remain in effect.

This also turns off validation of the gzip header CRC when
present.

This should only be used when a zlib or gzip stream has already
been checked, and repeated decompressions of the same stream no
longer need to be validated.

Correct the size of the inflate state in the comments.

Use TRIGGER_LEVEL in sse optimization too.

Fix _off64_t check for clang.

Remove dead code.

Fix shift of negative value in inflateMark().

Instruct make not to remove intermediate files.

Replace Z_NULL with NULL. Fix incorrect uses of NULL/Z_NULL.

Fix 'depend' target.

Fix debug build.

Update -DGEN_TREES_H output to match trees.h

Update dependency information for zutil.c

Variable h was initialized twice on first iteration.

Revert "Loop on write() calls in gzwrite.c in case of non-blocking I/O."

This reverts commit 4811aa74806df01ac7e5a28c7a6b42bc6795b7a4.

Reverted because the code returns incorrect data.
Needs to be investigated and re-implemented.

Fix missing insert_string change from merge conflict

Merge insert_string and bulk_insert_str.

** Partial merge of this commit, based on a8c94e9f5a3b9d3c62182bcf84e72304a3c1a6e5
Excludes changes to fill_window_sse.c, changes to fill_window_c() in deflate.c
and several unrelated changes in the commit.

Use bulk_insert_str when adding more than one byte.

** Partial merge of this commit, based on f8ad48ab17571c730b743c925f0e13ff83cf487f
Excluding changes to fill_window_sse.c, deflate.c and deflate_p.h

Use TRIGGER_LEVEL instead of hard-coded '6'.

** Only merged part of this commit, based on 9460967fcfde4d83574f82eabe2a2bc91a9c351a

Update .gitignore

ignore KDevelop statefiles

cpuid for msvc

Add a.exe to cleanup list.

local -> static

* local -> static
* Normalize and cleanup line-endings
* Fix warnings under Visual Studio.
* Whitespace cleanup

***
This patch has been edited to merge cleanly and to exclude type changes.
Based on 8d7a7c3b82c6e38734bd504dac800b148ab410d0 "Type Cleanup"

Merge pull request #74 from Majlen/develop

Using correct format specifiers

Using correct format specifiers

Fixes bug #73

Merge pull request #72 from mtl1979/duphead

Don't update prev if old head is same as new.

Don't update prev if old head is same as new.

Merge pull request #70 from jibsen/fix-win-clang

Do not define __builtin_ctzl for Clang on MSVC

Do not define __builtin_ctzl for Clang on MSVC

Clang targeting MSVC and Clang/C2 provide __builtin_ctzl.

Only run tests requiring zlib-compat if zlib-ng was compiled with it.

This also reveals that minigzip without zlib-compat is vulnerable
to one or more of the tests.

Fix spelling in travis config

Make travis test --without-optimizations and --without-new-strategies codepaths as well.

Both travis builds using configure should do zlib-compat

Travis should compile and test the zlib-compat code as well

Fix the remaining warnings due to conflicting defines of ZLIB_INTERNAL

Add test for CVE-2004-0797

CVE-2004-0797[0] occured when an error was detected but no action
was taken --- that is, execution was allowed to continue.

One of the tests for CVE-2005-2096 actually hit the code path that
was fixed in the patch for CVE-2004-0797.

This occured because all the fuzzing was done on zlib 1.2.1, and
zlib 1.2.2 fixed this bug but not the 2005 CVEs. It was detected by
running the test cases against zlib 1.2.2.

The relevant bits of the zlib 1.2.2 patch are [1] and [2].

[0] http://www.kb.cert.org/vuls/id/238678
[1] https://github.com/madler/zlib/commit/7a6955760ba950eb82f57929f8f6c9847c65f0af?diff=unified#diff-154f1240658ec1e9f5c90024002d749aR437
[2] https://github.com/madler/zlib/commit/7a6955760ba950eb82f57929f8f6c9847c65f0af?diff=unified#diff-327188edf18799ffbb5a51cc69f797e8R864

Signed-off-by: Daniel Axtens <dja@axtens.net>

Add test cases for CVE-2005-1849 and CVE-2005-2096

CVE-2005-1849[0] is an overflow of a fixed size buffer defined in
inftrees.h

CVE-2005-2096[1] is an overflow caused by insufficient input
validation of code trees[2].

This makes sure we don't accidentally reintroduce them.

zlib-1.2.1 was download and fuzz tested using AFL[3].  The crashing
cases were discovered. A patch for 1849 was then applied, and used to
determine which cases hit only that bug, and which ones hit 2096.

[0] http://seclists.org/bugtraq/2005/Jul/451
[1] http://seclists.org/bugtraq/2005/Jul/53
[2] https://bug299445.bugzilla.mozilla.org/attachment.cgi?id=188021
[3] http://lcamtuf.coredump.cx/afl/

Signed-off-by: Daniel Axtens <dja@axtens.net>

Add test for CVE-2002-0059

CVE-2002-0059 was a double free in inflation. [0]

This makes sure we don't accidentally reintroduce it.

zlib-1.1.3 was download and fuzz tested using AFL[1].
This crashing case (test.gz) was discovered, and using gdb it was
confirmed to be a double free in the expected place.

The test script looks for a normal error exit (status code 1),
and fails if any other code is returned.

[0] http://www.cvedetails.com/cve/CVE-2002-0059/
[1] http://lcamtuf.coredump.cx/afl/

Signed-off-by: Daniel Axtens <dja@axtens.net>

Add test for CVE-2003-0107

CVE-2003-0107[0] was a bug where zlib 1.1.4 failed to validate whether
arguments to gzprintf() fit within an internal buffer.

We should make sure that in refactoring we don't regress. Therefore,
build the sample code supplied in the original report [1], and check
if it crashes.

[0] http://www.cvedetails.com/cve/CVE-2003-0107/
[1] http://www.securityfocus.com/archive/1/312869

Signed-off-by: Daniel Axtens <dja@axtens.net>

Properly bail out when a test fails.

At an earlier point in development, shared libs were failing on Travis
but the overall build was succeeding. Stop that from happening by
bailing out harder.

Signed-off-by: Daniel Axtens <dja@axtens.net>

Move Makefile.in test-related parts into test/Makefile.in

Update configure to create test/Makefile.
Update /Makefile.in to call [dist]clean in test/

Signed-off-by: Daniel Axtens <dja@axtens.net>

Add support for internal attribute, the advantage of this over hidden
is for example that the compiler can safely assume that pointers to
functions declared internal can never be passed externally.
This allows the compiler to consider optimizations otherwise impossible.

Fix endianness-detection code on Solaris 11.

Solaris doesn't have sys/endian.h or endian.h, it has sys/byteorder.h,
which doesn't define BYTE_ORDER, it defines either _LITTLE_ENDIAN or
_BIG_ENDIAN.

Fix spelling of __DragonFly__ in crc32.c, found by @nemequ

Merge pull request #66 from mtl1979/mingw

Fixes for MinGW w64 and MSYS2.

Merge pull request #68 from mtl1979/import

Loop on write() calls in gzwrite.c in case of non-blocking I/O.

Merge pull request #67 from mtl1979/coverity

Fix Coverity warnings.

Loop on write() calls in gzwrite.c in case of non-blocking I/O.

Fix Coverity warnings.

Fix build under MSYS2.

Fix gzseek() problem on MinGW due to buggy _lseeki64 there.

Merge pull request #60 from mtl1979/cleanup

Cleanup

Use size_t for total_in and total_out.

Type cleanup.

Merge pull request #59 from Dead2/hacknslash2

Mix of fixes and improvements

Merge pull request #58 from mtl1979/import

Fix bug that accepted invalid zlib header when windowBits is zero.

Fix bug that accepted invalid zlib header when windowBits is zero.

When windowBits is zero, the size of the sliding window comes from
the zlib header.  The allowed values of the four-bit field are
0..7, but when windowBits is zero, values greater than 7 are
permitted and acted upon, resulting in large, mostly unused memory
allocations.  This fix rejects such invalid zlib headers.

Type cleanup.

Fix creating shared library under MSYS64.

Fix compilation on BSD

The endianness functions are in <sys/endian.h>, not <endian.h> as
they are in Linux.

minigzip: Use bigger buffer for writing.

Merge pull request #56 from Dead2/hacknslash2

Improvements for MSVC and backport fixes from zlib upstream and intel's fork

Remove inline and extern from function declaration.

Merge pull request #55 from jibsen/fix-uninitialized-var

Fix uninitialized variable

Fix uninitialized variable

Split insert_string_sse into separate file in arch folder.

Enable deflate_quick under MSVC.

minigzip: restructure gzread() to get rid of unnecessary temporary variables.

Fix for intels zlib fork, fixes their issues #2 and #3

Fix merge errors.

Fix typo.

Use a consistent and more modern approach to not use a parameter.

A remarkably creative and diverse set of approaches to letting the
compiler know that opaque was being used when it wasn't is changed
by this commit to the more standard (void)opaque.

 Conflicts:
zutil.c

Use UTF-8 for non-ASCII characters in ChangeLog.

Clean up portability for shifts and integer sizes.

Use const for static tree descriptions in deflate.

This is in order to permit shared memory for these structures.

 Conflicts:
deflate.h

Compile the gzopen_w() function when __CYGWIN__ defined.

 Conflicts:
gzlib.c
zlib.h

Assure that gzoffset() is correct when appending.

An open() with O_APPEND followed by an lseek() to determine the
position will return zero for a non-empty file, even though the
next write will start at the end of the file.  This commit works
around that by doing an lseek() to the end when appending.

Note in zlib.h that compress() uses Z_DEFAULT_COMPRESSION.

Do not initialize unsigned with -1 in compress.c uncompr.c.

Sun compiler complained.  Use (unsigned)0 - 1 instead.

Fix uncompress() to work on lengths more than a maximum unsigned.

 Conflicts:
uncompr.c

Fix compress() to work on lengths more than a maximum unsigned.

 Conflicts:
compress.c

Align deflateParams() and its documentation in zlib.h.

This updates the documentation to reflect the behavior of
deflateParams() when it is not able to compress all of the input
data provided so far due to insufficient output space.  It also
assures that data provided is compressed before the parameter
changes, even if at the beginning of the stream.

 Conflicts:
deflate.c

Improve speed of gzprintf() in transparent mode.

A few minor modifications done to help with conflicts.

Fix MSVC build and clean up unneccessary BSDLIBS leftovers

Clarify deflateReset() documentation.

It previously could have been misinterpreted to mean that parameter
changes after deflateInit2() would be reversed, which is not the
case.

(cherry picked from commit 5701f48cf5178355b1ec51614d44aa14251250e5)

Avoid uninitialized access by gzclose_w().

(cherry picked from commit c901a34c92c4aa74028f541a9773df726ce2b769)

 Conflicts:
deflate.c

Apply Mark Adlers fix to remove the need for -lbsd for reallocf, and
remove our detection of reallocf.

Fix inflateInit2() bug when windowBits is 16 or 32.

A windowBits value of 0, 16, or 32 gets the window bits from the
zlib header.  However there is no zlib header for 16, or for 32
when the input is gzip.  This commit sets the window bits for
inflate to 15 if a gzip stream is detected and windowBits was 16
or 32.

(cherry picked from commit 0db8fd371477f42c280ddeee29e6de092fabf948)

Add comment about not using windowBits of 8 for deflate().

(cherry picked from commit b56d1c62ee1ff9705026c94a780dd6e4577cda02)

Follow manpage update from zlib, adding licence to manpage.

Remedy Coverity warning. [Randers-Pehrson]

 Conflicts:
inflate.c