Boost C++ Libraries: Ticket #5908: iostreams gzip fails to handle optional extra fields in gzip header

attachment set

Sat, 17 Sep 2011 21:34:28 GMT

attachment → sample.txt.gz

gzip file with extra comment in header

attachment set

Sat, 17 Sep 2011 21:34:58 GMT

attachment → example.cpp

test case that demonstrates the problem (run ./example sample.txt.gz)

attachment set

Sat, 17 Sep 2011 21:35:41 GMT

attachment → iostreams-gzip.patch

patch that fixes the issue (patch -p0 < iostreams-gzip.patch from top level of the repo)

typedef.struct@… — Tue, 20 Dec 2011 21:27:31 GMT

Seems like this is still present in 1.48 and current SVN. To be clear, the sample file is not empty. The desired output would be for the test program to read "hello there", but it gets nothing. You can compare to gzip -dc. The supplied patch still works with current SVN.

typedef.struct@… — Tue, 20 Dec 2011 22:28:16 GMT

Also worth mentioning is that this worked in 1.40, but when the ability to support multiple compressed objects in 1 gzipped stream was introduced, this bug appeared.

attachment set

typedef.struct@… — Thu, 22 Dec 2011 21:25:07 GMT

attachment → iostreams-gzip_hdr_test.patch

second patch adding unit test to prevent future regressions.

typedef.struct@… — Thu, 22 Dec 2011 21:26:54 GMT

The test attached in the second patch (iostreams-gzip_hdr_test.patch) will fail with the current code, demonstrating the inability to parse certain types of RFC 1952 compliant gzip headers. Application of the first patch will cause the test to pass.

status changed

Jonathan Turkanis — Wed, 04 Jan 2012 04:52:01 GMT

status new → assigned

I have applied the patches to trunk.

status changed; resolution set

Daniel James — Sun, 18 Mar 2012 13:47:03 GMT

status assigned → closed
resolution → fixed

(In [77368]) Iostreams: Merge from trunk.

[74728] Eliminate unit_test_framework and BOOST_MESSAGE.
[76301] Fixes #5908.
[76941] Fix typos in comments. Fixes #6530.
[76973] Remove extra semi-colons, which break with trunk Boost.Test.
[77174] Fix parameters for filter::close. Fixes #6650.

status, severity, type, component, version, milestone changed; resolution deleted

Smithc195 — Fri, 06 Jan 2017 04:16:26 GMT

status closed → reopened
severity Problem → Not Applicable
type Bugs → Library Submissions
component iostreams → xpressive
version Boost 1.47.0 → Boost.Build-M3
milestone To Be Determined → Website 1.X
resolution fixed

Hi. Only wanted to ask a quick issue. Now i am eaadkebdkckdgfgd

status changed; resolution set

Daniel James — Fri, 06 Jan 2017 11:38:34 GMT

status reopened → closed
resolution → fixed

type, version, component, severity, milestone changed

Kohei Takahashi — Sun, 08 Jan 2017 14:33:22 GMT

type Library Submissions → Bugs
version Boost.Build-M3 → Boost 1.47.0
component xpressive → iostreams
severity Not Applicable → Problem
milestone Website 1.X → To Be Determined