Fix buffer overflow in inflateGetHeader()
authortb <tb@openbsd.org>
Tue, 9 Aug 2022 07:37:35 +0000 (07:37 +0000)
committertb <tb@openbsd.org>
Tue, 9 Aug 2022 07:37:35 +0000 (07:37 +0000)
commit9439c8ad6155442714447e70699f7ba977e232f5
treec54e4844a9328bcd428ba89bdb39ad5cb8f4be46
parent8272ea32c72a9b6974df8a91bd1bc3c0e2aa883f
Fix buffer overflow in inflateGetHeader()

This is the initial fix combined with a fix for a NULL deref introduced
in the initial fix.

ok millert, help from tj

commit eff308af425b67093bab25f80f1ae950166bece1
Author: Mark Adler <fork@madler.net>
Date:   Sat Jul 30 15:51:11 2022 -0700

    Fix a bug when getting a gzip header extra field with inflate().

    If the extra field was larger than the space the user provided with
    inflateGetHeader(), and if multiple calls of inflate() delivered
    the extra header data, then there could be a buffer overflow of the
    provided space. This commit assures that provided space is not
    exceeded.

https://github.com/madler/zlib/commit/eff308af425b67093bab25f80f1ae950166bece1

commit 1eb7682f845ac9e9bf9ae35bbfb3bad5dacbd91d (HEAD -> develop, origin/develop)
Author: Mark Adler <fork@madler.net>
Date:   Mon Aug 8 10:50:09 2022 -0700

    Fix extra field processing bug that dereferences NULL state->head.

    The recent commit to fix a gzip header extra field processing bug
    introduced the new bug fixed here.

https://github.com/madler/zlib/commit/1eb7682f845ac9e9bf9ae35bbfb3bad5dacbd91d
lib/libz/inflate.c