[Lzip-bug] memcheck errors in lzip-1.2-pre2

[John Reiser]

lzip-1.2-pre2 accesses memory outside of allocated regions.

$ rm -f foo foo.lz
$ cp COPYING foo
$ valgrind ./lzip -s23 foo
==4120== Memcheck, a memory error detector.
==4120== Copyright (C) 2002-2007, and GNU GPL'd, by Julian Seward et al.
==4120== Using LibVEX rev 1804, a library for dynamic binary translation.
==4120== Copyright (C) 2004-2007, and GNU GPL'd, by OpenWorks LLP.
==4120== Using valgrind-3.3.0, a dynamic binary instrumentation framework.
==4120== Copyright (C) 2000-2007, and GNU GPL'd, by Julian Seward et al.
==4120== For more details, rerun with: -v
==4120==
==4120== Invalid read of size 1
==4120==    at 0x804E31A: LZ_encoder::best_pair_sequence(int const*, State 
const&) (encoder.h:85)
==4120==    by 0x805179D: LZ_encoder::encode() (encoder.cc:451)
==4120==    by 0x8054BEE: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:239)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==  Address 0x441f027 is 1 bytes before a block of size 16,781,585 alloc'd
==4120==    at 0x4005E1E: operator new[](unsigned) (vg_replace_malloc.c:268)
==4120==    by 0x804FB74: LZ_encoder::LZ_encoder(File_header const&, int, int, 
int) (encoder.h:61)
==4120==    by 0x8054BE0: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:237)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==
==4120== Invalid read of size 1
==4120==    at 0x804E021: LZ_encoder::best_pair_sequence(int const*, State 
const&) (encoder.h:69)
==4120==    by 0x805179D: LZ_encoder::encode() (encoder.cc:451)
==4120==    by 0x8054BEE: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:239)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==  Address 0x441f027 is 1 bytes before a block of size 16,781,585 alloc'd
==4120==    at 0x4005E1E: operator new[](unsigned) (vg_replace_malloc.c:268)
==4120==    by 0x804FB74: LZ_encoder::LZ_encoder(File_header const&, int, int, 
int) (encoder.h:61)
==4120==    by 0x8054BE0: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:237)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==
==4120== Invalid read of size 1
==4120==    at 0x804E089: LZ_encoder::best_pair_sequence(int const*, State 
const&) (encoder.h:372)
==4120==    by 0x805179D: LZ_encoder::encode() (encoder.cc:451)
==4120==    by 0x8054BEE: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:239)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==  Address 0x441f027 is 1 bytes before a block of size 16,781,585 alloc'd
==4120==    at 0x4005E1E: operator new[](unsigned) (vg_replace_malloc.c:268)
==4120==    by 0x804FB74: LZ_encoder::LZ_encoder(File_header const&, int, int, 
int) (encoder.h:61)
==4120==    by 0x8054BE0: (anonymous namespace)::compress(int, int, (anonymous 
namespace)::lzma_options, Pretty_print const&) (main.cc:237)
==4120==    by 0x80555D6: main (main.cc:707)
==4120==
==4120== ERROR SUMMARY: 6 errors from 3 contexts (suppressed: 15 from 1)
==4120== malloc/free: in use at exit: 0 bytes in 0 blocks.
==4120== malloc/free: 14 allocs, 14 frees, 50,688,388 bytes allocated.

-- 
John Reiser