Boost C++ Libraries: Ticket #2783: bzip2_decompressor infinite loops with truncated input file

attachment set

Fri, 20 Feb 2009 09:43:01 GMT

attachment → hello.cc

attachment set

Fri, 20 Feb 2009 09:43:11 GMT

attachment → hello.bz2

status changed; resolution set

agander@… — Fri, 05 Jun 2009 20:01:02 GMT

status new → closed
resolution → invalid

Is this a valid test/error?

bzip2 reports that hello.bz2 is corrupt.

bzip2 -tvvv hello.bz2

hello.bz2:

[1: huff+mtf rt+rld {0x13cbede3, 0x13cbede3}]file ends unexpectedly

You can use the `bzip2recover' program to attempt to recover data from undamaged sections of corrupted files.

It works fine once recovered.

But then this isnt a Boost issue.

Tested on:

BOOST_VERSION: 103800 BOOST_LIB_VERSION: 1_38

gcc -v:- Using built-in specs. Target: x86_64-redhat-linux Configured with: ../configure --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-bootstrap --enable-shared --enable-threads=posix --enable-checking=release --with-system-zlib --enable-cxa_atexit --disable-libunwind-exceptions --enable-languages=c,c++,objc,obj-c++,java,fortran,ada --enable-java-awt=gtk --disable-dssi --enable-plugin --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-1.5.0.0/jre --enable-libgcj-multifile --enable-java-maintainer-mode --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --disable-libjava-multilib --with-cpu=generic --build=x86_64-redhat-linux Thread model: posix gcc version 4.3.2 20081105 (Red Hat 4.3.2-7) (GCC)

Linux 2.6.27.24-170.2.68.fc10.x86_64 #1 SMP x86_64 GNU/Linux

markboost@… — Fri, 05 Jun 2009 20:55:41 GMT

I disagree. (As does Jonathan Turkanis - http://lists.boost.org/boost-users/2005/10/14362.php).

The correct behaviour is to throw an exception (probably boost::iostreams::data_error) (like the gzip filter does) - infinite looping with invalid input really isn't good. Just like bzip2 reports the file is corrupt, so should this filter.

status changed; resolution deleted

agander@… — Fri, 05 Jun 2009 21:32:13 GMT

status closed → reopened
resolution invalid

Ok, fair point. (Thats an embarassing start...)

status changed; resolution set

Steven Watanabe — Tue, 15 Jun 2010 02:52:45 GMT

status reopened → closed
resolution → fixed

(In [62965]) Throw an exception on an unexpected end of file in bzip2_decompressor instead of going into an infinite loop. Fixes #2783.