]> wimlib.net Git - wimlib/blobdiff - src/lzx-decompress.c
{lzx,lzms-decompress.c}: Allocate context with DECODE_TABLE_ALIGNMENT
[wimlib] / src / lzx-decompress.c
index d25d188b08b1e8fa6320f17b47127fa617d5198c..d259eeedc38ae31b6c7156fc0b1599f3b2af8d90 100644 (file)
@@ -7,7 +7,7 @@
  */
 
 /*
- * Copyright (C) 2012, 2013 Eric Biggers
+ * Copyright (C) 2012, 2013, 2014 Eric Biggers
  *
  * This file is part of wimlib, a library for working with WIM files.
  *
  */
 
 /*
- * LZX is a LZ77 and Huffman-code based compression format that has many
+ * LZX is an LZ77 and Huffman-code based compression format that has many
  * similarities to the DEFLATE format used in zlib.  The compression ratio is as
- * good or better than DEFLATE.  However, in WIM files only up to 32768 bytes of
- * data can ever compressed be in the same LZX block, so a .tar.gz file could
- * potentially be smaller than a WIM file that uses LZX compression because it
- * can use a larger LZ77 window size.
+ * good or better than DEFLATE.
  *
  * Some notes on the LZX compression format as used in Windows Imaging (WIM)
  * files:
  *
  * A compressed WIM resource consists of a table of chunk offsets followed by
  * the compressed chunks themselves.  All compressed chunks except possibly the
- * last decompress to WIM_CHUNK_SIZE (= 32768) bytes.  This is quite similar to
- * the cabinet (.cab) file format, but they are not the same.  According to the
- * cabinet format documentation, the LZX block size is independent from the
- * CFDATA blocks, and a LZX block may span several CFDATA blocks.  However, in
- * WIMs, LZX blocks do not appear to ever span multiple WIM chunks.  Note that
- * this means any WIM chunk may be decompressed or compressed independently from
- * any other chunk, which is convenient.
- *
- * A LZX compressed WIM chunk contains one or more LZX blocks of the aligned,
+ * last decompress to a fixed number of bytes, by default 32768.  This is quite
+ * similar to the cabinet (.cab) file format, but they are not the same.
+ * According to the cabinet format documentation, the LZX block size is
+ * independent from the CFDATA blocks, and an LZX block may span several CFDATA
+ * blocks.  However, in WIMs, LZX blocks do not appear to ever span multiple WIM
+ * chunks.  Note that this means any WIM chunk may be decompressed or compressed
+ * independently from any other chunk, which allows random access.
+ *
+ * An LZX compressed WIM chunk contains one or more LZX blocks of the aligned,
  * verbatim, or uncompressed block types.  For aligned and verbatim blocks, the
  * size of the block in uncompressed bytes is specified by a bit following the 3
  * bits that specify the block type, possibly followed by an additional 16 bits.
- * '1' means to use the default block size (equal to 32768, the size of a WIM
- * chunk--- and this seems to only be valid for the first LZX block in a WIM
- * chunk), while '0' means that the block size is provided by the next 16 bits.
+ * '1' means to use the default block size (equal to 32768, the default size of
+ * a WIM chunk), while '0' means that the block size is provided by the next 16
+ * bits.
  *
  * The cabinet format, as documented, allows for the possibility that a
  * compressed CFDATA chunk is up to 6144 bytes larger than the data it
  * defined in the specification.
  *
  * The LZX document states that aligned offset blocks have their aligned offset
- * huffman tree AFTER the main and length trees. The implementation suggests
+ * Huffman tree AFTER the main and length trees. The implementation suggests
  * that the aligned offset tree is BEFORE the main and length trees.
  *
  * The LZX document decoding algorithm states that, in an aligned offset block,
  * if an extra_bits value is 1, 2 or 3, then that number of bits should be read
  * and the result added to the match offset. This is correct for 1 and 2, but
- * not 3, where just a huffman symbol (using the aligned tree) should be read.
+ * not 3, where just a Huffman symbol (using the aligned tree) should be read.
  *
  * Regarding the E8 preprocessing, the LZX document states 'No translation may
  * be performed on the last 6 bytes of the input block'. This is correct.
  * would cause the next four bytes to be modified, at least one of which would
  * be in the last 6 bytes, which is not allowed according to the spec.
  *
- * The specification states that the huffman trees must always contain at least
+ * The specification states that the Huffman trees must always contain at least
  * one element. However, many CAB files contain blocks where the length tree is
  * completely empty (because there are no matches), and this is expected to
  * succeed.
 #endif
 
 #include "wimlib.h"
-#include "wimlib/decompress.h"
+#include "wimlib/decompressor_ops.h"
+#include "wimlib/decompress_common.h"
 #include "wimlib/lzx.h"
 #include "wimlib/util.h"
 
 /* Huffman decoding tables and maps from symbols to code lengths. */
 struct lzx_tables {
 
-       u16 maintree_decode_table[(1 << LZX_MAINTREE_TABLEBITS) +
-                                       (LZX_MAINTREE_NUM_SYMBOLS * 2)]
+       u16 maintree_decode_table[(1 << LZX_MAINCODE_TABLEBITS) +
+                                       (LZX_MAINCODE_MAX_NUM_SYMBOLS * 2)]
                                        _aligned_attribute(DECODE_TABLE_ALIGNMENT);
-       u8 maintree_lens[LZX_MAINTREE_NUM_SYMBOLS];
+       u8 maintree_lens[LZX_MAINCODE_MAX_NUM_SYMBOLS];
 
 
-       u16 lentree_decode_table[(1 << LZX_LENTREE_TABLEBITS) +
-                                       (LZX_LENTREE_NUM_SYMBOLS * 2)]
+       u16 lentree_decode_table[(1 << LZX_LENCODE_TABLEBITS) +
+                                       (LZX_LENCODE_NUM_SYMBOLS * 2)]
                                        _aligned_attribute(DECODE_TABLE_ALIGNMENT);
-       u8 lentree_lens[LZX_LENTREE_NUM_SYMBOLS];
+       u8 lentree_lens[LZX_LENCODE_NUM_SYMBOLS];
 
 
-       u16 alignedtree_decode_table[(1 << LZX_ALIGNEDTREE_TABLEBITS) +
-                                       (LZX_ALIGNEDTREE_NUM_SYMBOLS * 2)]
+       u16 alignedtree_decode_table[(1 << LZX_ALIGNEDCODE_TABLEBITS) +
+                                       (LZX_ALIGNEDCODE_NUM_SYMBOLS * 2)]
                                        _aligned_attribute(DECODE_TABLE_ALIGNMENT);
-       u8 alignedtree_lens[LZX_ALIGNEDTREE_NUM_SYMBOLS];
+       u8 alignedtree_lens[LZX_ALIGNEDCODE_NUM_SYMBOLS];
 } _aligned_attribute(DECODE_TABLE_ALIGNMENT);
 
+struct lzx_decompressor {
+       u32 max_window_size;
+       unsigned num_main_syms;
+       struct lzx_tables tables;
+};
 
 /*
  * Reads a Huffman-encoded symbol using the pre-tree.
  */
-static inline int
+static inline u16
 read_huffsym_using_pretree(struct input_bitstream *istream,
-                          const u16 pretree_decode_table[],
-                          const u8 pretree_lens[], unsigned *n)
+                          const u16 pretree_decode_table[])
 {
-       return read_huffsym(istream, pretree_decode_table, pretree_lens,
-                           LZX_PRETREE_NUM_SYMBOLS, LZX_PRETREE_TABLEBITS, n,
-                           LZX_MAX_CODEWORD_LEN);
+       return read_huffsym(istream, pretree_decode_table,
+                           LZX_PRECODE_TABLEBITS, LZX_MAX_PRE_CODEWORD_LEN);
 }
 
 /* Reads a Huffman-encoded symbol using the main tree. */
-static inline int
+static inline u16
 read_huffsym_using_maintree(struct input_bitstream *istream,
-                           const struct lzx_tables *tables,
-                           unsigned *n)
+                           const struct lzx_tables *tables)
 {
        return read_huffsym(istream, tables->maintree_decode_table,
-                           tables->maintree_lens, LZX_MAINTREE_NUM_SYMBOLS,
-                           LZX_MAINTREE_TABLEBITS, n, LZX_MAX_CODEWORD_LEN);
+                           LZX_MAINCODE_TABLEBITS, LZX_MAX_MAIN_CODEWORD_LEN);
 }
 
 /* Reads a Huffman-encoded symbol using the length tree. */
-static inline int
+static inline u16
 read_huffsym_using_lentree(struct input_bitstream *istream,
-                          const struct lzx_tables *tables,
-                          unsigned *n)
+                          const struct lzx_tables *tables)
 {
        return read_huffsym(istream, tables->lentree_decode_table,
-                           tables->lentree_lens, LZX_LENTREE_NUM_SYMBOLS,
-                           LZX_LENTREE_TABLEBITS, n, LZX_MAX_CODEWORD_LEN);
+                           LZX_LENCODE_TABLEBITS, LZX_MAX_LEN_CODEWORD_LEN);
 }
 
 /* Reads a Huffman-encoded symbol using the aligned offset tree. */
-static inline int
+static inline u16
 read_huffsym_using_alignedtree(struct input_bitstream *istream,
-                              const struct lzx_tables *tables,
-                              unsigned *n)
+                              const struct lzx_tables *tables)
 {
        return read_huffsym(istream, tables->alignedtree_decode_table,
-                           tables->alignedtree_lens,
-                           LZX_ALIGNEDTREE_NUM_SYMBOLS,
-                           LZX_ALIGNEDTREE_TABLEBITS, n, 8);
+                           LZX_ALIGNEDCODE_TABLEBITS, LZX_MAX_ALIGNED_CODEWORD_LEN);
 }
 
 /*
@@ -191,10 +185,10 @@ read_huffsym_using_alignedtree(struct input_bitstream *istream,
  * code length values from the input.
  *
  * @istream:   The bit stream for the input.  It is positioned on the beginning
- *                     of the pretree for the code length values.
+ *                     of the pretree for the code length values.
  * @lens:      An array that contains the length values from the previous time
- *                     the code lengths for this Huffman tree were read, or all
- *                     0's if this is the first time.
+ *                     the code lengths for this Huffman tree were read, or all
+ *                     0's if this is the first time.
  * @num_lens:  Number of length values to decode and return.
  *
  */
@@ -203,31 +197,27 @@ lzx_read_code_lens(struct input_bitstream *istream, u8 lens[],
                   unsigned num_lens)
 {
        /* Declare the decoding table and length table for the pretree. */
-       u16 pretree_decode_table[(1 << LZX_PRETREE_TABLEBITS) +
-                                       (LZX_PRETREE_NUM_SYMBOLS * 2)]
+       u16 pretree_decode_table[(1 << LZX_PRECODE_TABLEBITS) +
+                                       (LZX_PRECODE_NUM_SYMBOLS * 2)]
                                        _aligned_attribute(DECODE_TABLE_ALIGNMENT);
-       u8 pretree_lens[LZX_PRETREE_NUM_SYMBOLS];
+       u8 pretree_lens[LZX_PRECODE_NUM_SYMBOLS];
        unsigned i;
-       unsigned len;
        int ret;
 
        /* Read the code lengths of the pretree codes.  There are 20 lengths of
         * 4 bits each. */
-       for (i = 0; i < LZX_PRETREE_NUM_SYMBOLS; i++) {
-               ret = bitstream_read_bits(istream, LZX_PRETREE_ELEMENT_SIZE,
-                                         &len);
-               if (ret != 0)
-                       return ret;
-               pretree_lens[i] = len;
+       for (i = 0; i < LZX_PRECODE_NUM_SYMBOLS; i++) {
+               pretree_lens[i] = bitstream_read_bits(istream,
+                                                     LZX_PRECODE_ELEMENT_SIZE);
        }
 
        /* Make the decoding table for the pretree. */
        ret = make_huffman_decode_table(pretree_decode_table,
-                                       LZX_PRETREE_NUM_SYMBOLS,
-                                       LZX_PRETREE_TABLEBITS,
+                                       LZX_PRECODE_NUM_SYMBOLS,
+                                       LZX_PRECODE_TABLEBITS,
                                        pretree_lens,
-                                       LZX_MAX_CODEWORD_LEN);
-       if (ret != 0)
+                                       LZX_MAX_PRE_CODEWORD_LEN);
+       if (ret)
                return ret;
 
        /* Pointer past the last length value that needs to be filled in. */
@@ -242,20 +232,16 @@ lzx_read_code_lens(struct input_bitstream *istream, u8 lens[],
                 * the next lengths are all equal to the next symbol in the
                 * input. */
                unsigned tree_code;
-               unsigned num_zeroes;
+               u32 num_zeroes;
                unsigned code;
-               unsigned num_same;
+               u32 num_same;
                signed char value;
 
-               ret = read_huffsym_using_pretree(istream, pretree_decode_table,
-                                                pretree_lens, &tree_code);
-               if (ret != 0)
-                       return ret;
+               tree_code = read_huffsym_using_pretree(istream,
+                                                      pretree_decode_table);
                switch (tree_code) {
                case 17: /* Run of 0's */
-                       ret = bitstream_read_bits(istream, 4, &num_zeroes);
-                       if (ret != 0)
-                               return ret;
+                       num_zeroes = bitstream_read_bits(istream, 4);
                        num_zeroes += 4;
                        while (num_zeroes--) {
                                *lens = 0;
@@ -264,9 +250,7 @@ lzx_read_code_lens(struct input_bitstream *istream, u8 lens[],
                        }
                        break;
                case 18: /* Longer run of 0's */
-                       ret = bitstream_read_bits(istream, 5, &num_zeroes);
-                       if (ret != 0)
-                               return ret;
+                       num_zeroes = bitstream_read_bits(istream, 5);
                        num_zeroes += 20;
                        while (num_zeroes--) {
                                *lens = 0;
@@ -275,16 +259,10 @@ lzx_read_code_lens(struct input_bitstream *istream, u8 lens[],
                        }
                        break;
                case 19: /* Run of identical lengths */
-                       ret = bitstream_read_bits(istream, 1, &num_same);
-                       if (ret != 0)
-                               return ret;
+                       num_same = bitstream_read_bits(istream, 1);
                        num_same += 4;
-                       ret = read_huffsym_using_pretree(istream,
-                                                        pretree_decode_table,
-                                                        pretree_lens,
-                                                        &code);
-                       if (ret != 0)
-                               return ret;
+                       code = read_huffsym_using_pretree(istream,
+                                                         pretree_decode_table);
                        value = (signed char)*lens - (signed char)code;
                        if (value < 0)
                                value += 17;
@@ -311,18 +289,20 @@ lzx_read_code_lens(struct input_bitstream *istream, u8 lens[],
  *
  * @istream:           The input bitstream.
  * @block_size_ret:    A pointer to an int into which the size of the block,
- *                             in bytes, will be returned.
+ *                             in bytes, will be returned.
  * @block_type_ret:    A pointer to an int into which the type of the block
- *                             (LZX_BLOCKTYPE_*) will be returned.
- * @tables:            A pointer to a lzx_tables structure in which the
- *                             main tree, the length tree, and possibly the
- *                             aligned offset tree will be constructed.
+ *                             (LZX_BLOCKTYPE_*) will be returned.
+ * @tables:            A pointer to an lzx_tables structure in which the
+ *                             main tree, the length tree, and possibly the
+ *                             aligned offset tree will be constructed.
  * @queue:     A pointer to the least-recently-used queue into which
- *                     R0, R1, and R2 will be written (only for uncompressed
- *                     blocks, which contain this information in the header)
+ *                     R0, R1, and R2 will be written (only for uncompressed
+ *                     blocks, which contain this information in the header)
  */
 static int
 lzx_read_block_header(struct input_bitstream *istream,
+                     unsigned num_main_syms,
+                     unsigned max_window_size,
                      unsigned *block_size_ret,
                      unsigned *block_type_ret,
                      struct lzx_tables *tables,
@@ -331,32 +311,33 @@ lzx_read_block_header(struct input_bitstream *istream,
        int ret;
        unsigned block_type;
        unsigned block_size;
-       unsigned s;
-       unsigned i;
-       unsigned len;
 
-       ret = bitstream_ensure_bits(istream, LZX_BLOCKTYPE_NBITS + 1);
-       if (ret) {
-               LZX_DEBUG("LZX input stream overrun");
-               return ret;
-       }
+       bitstream_ensure_bits(istream, 4);
 
        /* The first three bits tell us what kind of block it is, and are one
         * of the LZX_BLOCKTYPE_* values.  */
-       block_type = bitstream_read_bits_nocheck(istream, LZX_BLOCKTYPE_NBITS);
-
-       /* The next bit indicates whether the block size is the default (32768),
-        * indicated by a 1 bit, or whether the block size is given by the next
-        * 16 bits, indicated by a 0 bit. */
-       s = bitstream_read_bits_nocheck(istream, 1);
+       block_type = bitstream_pop_bits(istream, 3);
 
-       if (s) {
-               block_size = 32768;
+       /* Read the block size.  This mirrors the behavior
+        * lzx_write_compressed_block() in lzx-compress.c; see that for more
+        * details.  */
+       if (bitstream_pop_bits(istream, 1)) {
+               block_size = LZX_DEFAULT_BLOCK_SIZE;
        } else {
-               ret = bitstream_read_bits(istream, LZX_BLOCKSIZE_NBITS, &block_size);
-               if (ret)
-                       return ret;
-               block_size = le16_to_cpu(block_size);
+               u32 tmp;
+               block_size = 0;
+
+               tmp = bitstream_read_bits(istream, 8);
+               block_size |= tmp;
+               tmp = bitstream_read_bits(istream, 8);
+               block_size <<= 8;
+               block_size |= tmp;
+
+               if (max_window_size >= 65536) {
+                       tmp = bitstream_read_bits(istream, 8);
+                       block_size <<= 8;
+                       block_size |= tmp;
+               }
        }
 
        switch (block_type) {
@@ -364,21 +345,18 @@ lzx_read_block_header(struct input_bitstream *istream,
                /* Read the path lengths for the elements of the aligned tree,
                 * then build it. */
 
-               for (i = 0; i < LZX_ALIGNEDTREE_NUM_SYMBOLS; i++) {
-                       ret = bitstream_read_bits(istream,
-                                                 LZX_ALIGNEDTREE_ELEMENT_SIZE,
-                                                 &len);
-                       if (ret)
-                               return ret;
-                       tables->alignedtree_lens[i] = len;
+               for (unsigned i = 0; i < LZX_ALIGNEDCODE_NUM_SYMBOLS; i++) {
+                       tables->alignedtree_lens[i] =
+                               bitstream_read_bits(istream,
+                                                   LZX_ALIGNEDCODE_ELEMENT_SIZE);
                }
 
                LZX_DEBUG("Building the aligned tree.");
                ret = make_huffman_decode_table(tables->alignedtree_decode_table,
-                                               LZX_ALIGNEDTREE_NUM_SYMBOLS,
-                                               LZX_ALIGNEDTREE_TABLEBITS,
+                                               LZX_ALIGNEDCODE_NUM_SYMBOLS,
+                                               LZX_ALIGNEDCODE_TABLEBITS,
                                                tables->alignedtree_lens,
-                                               8);
+                                               LZX_MAX_ALIGNED_CODEWORD_LEN);
                if (ret) {
                        LZX_DEBUG("Failed to make the decode table for the "
                                  "aligned offset tree");
@@ -407,10 +385,10 @@ lzx_read_block_header(struct input_bitstream *istream,
                 * tree. */
                LZX_DEBUG("Reading path lengths for remaining elements of "
                          "main tree (%d elements).",
-                         LZX_MAINTREE_NUM_SYMBOLS - LZX_NUM_CHARS);
+                         num_main_syms - LZX_NUM_CHARS);
                ret = lzx_read_code_lens(istream,
                                         tables->maintree_lens + LZX_NUM_CHARS,
-                                        LZX_MAINTREE_NUM_SYMBOLS - LZX_NUM_CHARS);
+                                        num_main_syms - LZX_NUM_CHARS);
                if (ret) {
                        LZX_DEBUG("Failed to read the path lengths for the "
                                  "remaining elements of the main tree");
@@ -421,10 +399,10 @@ lzx_read_block_header(struct input_bitstream *istream,
                          "table for the main tree.");
 
                ret = make_huffman_decode_table(tables->maintree_decode_table,
-                                               LZX_MAINTREE_NUM_SYMBOLS,
-                                               LZX_MAINTREE_TABLEBITS,
+                                               num_main_syms,
+                                               LZX_MAINCODE_TABLEBITS,
                                                tables->maintree_lens,
-                                               LZX_MAX_CODEWORD_LEN);
+                                               LZX_MAX_MAIN_CODEWORD_LEN);
                if (ret) {
                        LZX_DEBUG("Failed to make the decode "
                                  "table for the main tree");
@@ -433,7 +411,7 @@ lzx_read_block_header(struct input_bitstream *istream,
 
                LZX_DEBUG("Reading path lengths for the length tree.");
                ret = lzx_read_code_lens(istream, tables->lentree_lens,
-                                        LZX_LENTREE_NUM_SYMBOLS);
+                                        LZX_LENCODE_NUM_SYMBOLS);
                if (ret) {
                        LZX_DEBUG("Failed to read the path "
                                  "lengths for the length tree");
@@ -442,10 +420,10 @@ lzx_read_block_header(struct input_bitstream *istream,
 
                LZX_DEBUG("Building the length tree.");
                ret = make_huffman_decode_table(tables->lentree_decode_table,
-                                               LZX_LENTREE_NUM_SYMBOLS,
-                                               LZX_LENTREE_TABLEBITS,
+                                               LZX_LENCODE_NUM_SYMBOLS,
+                                               LZX_LENCODE_TABLEBITS,
                                                tables->lentree_lens,
-                                               LZX_MAX_CODEWORD_LEN);
+                                               LZX_MAX_LEN_CODEWORD_LEN);
                if (ret) {
                        LZX_DEBUG("Failed to build the length Huffman tree");
                        return ret;
@@ -478,9 +456,9 @@ lzx_read_block_header(struct input_bitstream *istream,
                        istream->bitsleft = 0;
                        istream->bitbuf = 0;
                }
-               queue->R0 = le32_to_cpu(*(u32*)(istream->data + 0));
-               queue->R1 = le32_to_cpu(*(u32*)(istream->data + 4));
-               queue->R2 = le32_to_cpu(*(u32*)(istream->data + 8));
+               queue->R[0] = le32_to_cpu(*(le32*)(istream->data + 0));
+               queue->R[1] = le32_to_cpu(*(le32*)(istream->data + 4));
+               queue->R[2] = le32_to_cpu(*(le32*)(istream->data + 8));
                istream->data += 12;
                istream->data_bytes_left -= 12;
                /* The uncompressed data of this block directly follows and will
@@ -505,14 +483,14 @@ lzx_read_block_header(struct input_bitstream *istream,
  *                     tree.
  *
  * @block_type:                The type of the block (LZX_BLOCKTYPE_ALIGNED or
- *                     LZX_BLOCKTYPE_VERBATIM)
+ *                     LZX_BLOCKTYPE_VERBATIM)
  *
  * @bytes_remaining:   The amount of uncompressed data remaining to be
- *                     uncompressed in this block.  It is an error if the match
- *                     is longer than this number.
+ *                     uncompressed in this block.  It is an error if the match
+ *                     is longer than this number.
  *
  * @window:            A pointer to the window into which the uncompressed
- *                     data is being written.
+ *                     data is being written.
  *
  * @window_pos:                The current byte offset in the window.
  *
@@ -526,9 +504,9 @@ lzx_read_block_header(struct input_bitstream *istream,
  *
  * Returns the length of the match, or a negative number on error.  The possible
  * error cases are:
- *     - Match would exceed the amount of data remaining to be uncompressed.
- *     - Match refers to data before the window.
- *     - The input bitstream ended unexpectedly.
+ *     - Match would exceed the amount of data remaining to be uncompressed.
+ *     - Match refers to data before the window.
+ *     - The input bitstream ended unexpectedly.
  */
 static int
 lzx_decode_match(unsigned main_element, int block_type,
@@ -542,12 +520,10 @@ lzx_decode_match(unsigned main_element, int block_type,
        unsigned position_slot;
        unsigned match_len;
        unsigned match_offset;
-       unsigned additional_len;
        unsigned num_extra_bits;
-       unsigned verbatim_bits;
-       unsigned aligned_bits;
+       u32 verbatim_bits;
+       u32 aligned_bits;
        unsigned i;
-       int ret;
        u8 *match_dest;
        u8 *match_src;
 
@@ -561,38 +537,26 @@ lzx_decode_match(unsigned main_element, int block_type,
        position_slot = main_element >> 3;
 
        /* If the length_header is less than LZX_NUM_PRIMARY_LENS (= 7), it
-        * gives the match length as the offset from LZX_MIN_MATCH.  Otherwise,
-        * the length is given by an additional symbol encoded using the length
-        * tree, offset by 9 (LZX_MIN_MATCH + LZX_NUM_PRIMARY_LENS) */
-       match_len = LZX_MIN_MATCH + length_header;
-       if (length_header == LZX_NUM_PRIMARY_LENS) {
-               ret = read_huffsym_using_lentree(istream, tables,
-                                                &additional_len);
-               if (ret != 0)
-                       return ret;
-               match_len += additional_len;
-       }
-
+        * gives the match length as the offset from LZX_MIN_MATCH_LEN.
+        * Otherwise, the length is given by an additional symbol encoded using
+        * the length tree, offset by 9 (LZX_MIN_MATCH_LEN +
+        * LZX_NUM_PRIMARY_LENS) */
+       match_len = LZX_MIN_MATCH_LEN + length_header;
+       if (length_header == LZX_NUM_PRIMARY_LENS)
+               match_len += read_huffsym_using_lentree(istream, tables);
 
        /* If the position_slot is 0, 1, or 2, the match offset is retrieved
         * from the LRU queue.  Otherwise, the match offset is not in the LRU
         * queue. */
-       switch (position_slot) {
-       case 0:
-               match_offset = queue->R0;
-               break;
-       case 1:
-               match_offset = queue->R1;
-               swap(queue->R0, queue->R1);
-               break;
-       case 2:
-               /* The queue doesn't work quite the same as a real LRU queue,
-                * since using the R2 offset doesn't bump the R1 offset down to
-                * R2. */
-               match_offset = queue->R2;
-               swap(queue->R0, queue->R2);
-               break;
-       default:
+       if (position_slot <= 2) {
+               /* Note: This isn't a real LRU queue, since using the R2 offset
+                * doesn't bump the R1 offset down to R2.  This quirk allows all
+                * 3 recent offsets to be handled by the same code.  (For R0,
+                * the swap is a no-op.)  */
+               match_offset = queue->R[position_slot];
+               queue->R[position_slot] = queue->R[0];
+               queue->R[0] = match_offset;
+       } else {
                /* Otherwise, the offset was not encoded as one the offsets in
                 * the queue.  Depending on the position slot, there is a
                 * certain number of extra bits that need to be read to fully
@@ -615,52 +579,41 @@ lzx_decode_match(unsigned main_element, int block_type,
                         * equal to 3.  (Note that in the case with
                         * num_extra_bits == 3, the assignment to verbatim_bits
                         * will just set it to 0. ) */
-                       ret = bitstream_read_bits(istream, num_extra_bits - 3,
-                                                 &verbatim_bits);
-                       if (ret != 0)
-                               return ret;
-
+                       verbatim_bits = bitstream_read_bits(istream,
+                                                           num_extra_bits - 3);
                        verbatim_bits <<= 3;
-
-                       ret = read_huffsym_using_alignedtree(istream, tables,
-                                                            &aligned_bits);
-                       if (ret != 0)
-                               return ret;
+                       aligned_bits = read_huffsym_using_alignedtree(istream,
+                                                                     tables);
                } else {
                        /* For non-aligned blocks, or for aligned blocks with
                         * less than 3 extra bits, the extra bits are added
                         * directly to the match offset, and the correction for
                         * the alignment is taken to be 0. */
-                       ret = bitstream_read_bits(istream, num_extra_bits,
-                                                 &verbatim_bits);
-                       if (ret != 0)
-                               return ret;
-
+                       verbatim_bits = bitstream_read_bits(istream, num_extra_bits);
                        aligned_bits = 0;
                }
 
                /* Calculate the match offset. */
                match_offset = lzx_position_base[position_slot] +
-                              verbatim_bits + aligned_bits - 2;
+                              verbatim_bits + aligned_bits - LZX_OFFSET_OFFSET;
 
                /* Update the LRU queue. */
-               queue->R2 = queue->R1;
-               queue->R1 = queue->R0;
-               queue->R0 = match_offset;
-               break;
+               queue->R[2] = queue->R[1];
+               queue->R[1] = queue->R[0];
+               queue->R[0] = match_offset;
        }
 
        /* Verify that the match is in the bounds of the part of the window
         * currently in use, then copy the source of the match to the current
         * position. */
 
-       if (match_len > bytes_remaining) {
+       if (unlikely(match_len > bytes_remaining)) {
                LZX_DEBUG("Match of length %u bytes overflows "
                          "uncompressed block size", match_len);
                return -1;
        }
 
-       if (match_offset > window_pos) {
+       if (unlikely(match_offset > window_pos)) {
                LZX_DEBUG("Match of length %u bytes references "
                          "data before window (match_offset = %u, "
                          "window_pos = %u)",
@@ -690,73 +643,18 @@ lzx_decode_match(unsigned main_element, int block_type,
        return match_len;
 }
 
-static void
-undo_call_insn_translation(u32 *call_insn_target, int input_pos,
-                          s32 file_size)
-{
-       s32 abs_offset;
-       s32 rel_offset;
-
-       abs_offset = le32_to_cpu(*call_insn_target);
-       if (abs_offset >= -input_pos && abs_offset < file_size) {
-               if (abs_offset >= 0) {
-                       /* "good translation" */
-                       rel_offset = abs_offset - input_pos;
-               } else {
-                       /* "compensating translation" */
-                       rel_offset = abs_offset + file_size;
-               }
-               *call_insn_target = cpu_to_le32(rel_offset);
-       }
-}
-
-/* Undo the 'E8' preprocessing, where the targets of x86 CALL instructions were
- * changed from relative offsets to absolute offsets.
- *
- * Note that this call instruction preprocessing can and will be used on any
- * data even if it is not actually x86 machine code.  In fact, this type of
- * preprocessing appears to always be used in LZX-compressed resources in WIM
- * files; there is no bit to indicate whether it is used or not, unlike in the
- * LZX compressed format as used in cabinet files, where a bit is reserved for
- * that purpose.
- *
- * Call instruction preprocessing is disabled in the last 6 bytes of the
- * uncompressed data, which really means the 5-byte call instruction cannot
- * start in the last 10 bytes of the uncompressed data.  This is one of the
- * errors in the LZX documentation.
- *
- * Call instruction preprocessing does not appear to be disabled after the
- * 32768th chunk of a WIM stream, which is apparently is yet another difference
- * from the LZX compression used in cabinet files.
- *
- * Call instruction processing is supposed to take the file size as a parameter,
- * as it is used in calculating the translated jump targets.  But in WIM files,
- * this file size is always the same (LZX_WIM_MAGIC_FILESIZE == 12000000).*/
-static void
-undo_call_insn_preprocessing(u8 uncompressed_data[], int uncompressed_data_len)
-{
-       for (int i = 0; i < uncompressed_data_len - 10; i++) {
-               if (uncompressed_data[i] == 0xe8) {
-                       undo_call_insn_translation((u32*)&uncompressed_data[i + 1],
-                                                  i,
-                                                  LZX_WIM_MAGIC_FILESIZE);
-                       i += 4;
-               }
-       }
-}
-
 /*
- * Decompresses a LZX-compressed block of data from which the header has already
+ * Decompresses an LZX-compressed block of data from which the header has already
  * been read.
  *
  * @block_type:        The type of the block (LZX_BLOCKTYPE_VERBATIM or
- *             LZX_BLOCKTYPE_ALIGNED)
+ *             LZX_BLOCKTYPE_ALIGNED)
  * @block_size:        The size of the block, in bytes.
  * @window:    Pointer to the decompression window.
  * @window_pos:        The current position in the window.  Will be 0 for the first
- *                     block.
+ *                     block.
  * @tables:    The Huffman decoding tables for the block (main, length, and
- *                     aligned offset, the latter only for LZX_BLOCKTYPE_ALIGNED)
+ *                     aligned offset, the latter only for LZX_BLOCKTYPE_ALIGNED)
  * @queue:     The least-recently-used queue for match offsets.
  * @istream:   The input bitstream for the compressed literals.
  */
@@ -770,21 +668,16 @@ lzx_decompress_block(int block_type, unsigned block_size,
 {
        unsigned main_element;
        unsigned end;
-       int ret;
        int match_len;
 
        end = window_pos + block_size;
        while (window_pos < end) {
-               ret = read_huffsym_using_maintree(istream, tables,
-                                                 &main_element);
-               if (ret)
-                       return ret;
-
+               main_element = read_huffsym_using_maintree(istream, tables);
                if (main_element < LZX_NUM_CHARS) {
                        /* literal: 0 to LZX_NUM_CHARS - 1 */
                        window[window_pos++] = main_element;
                } else {
-                       /* match: LZX_NUM_CHARS to LZX_MAINTREE_NUM_SYMBOLS - 1 */
+                       /* match: LZX_NUM_CHARS to num_main_syms - 1 */
                        match_len = lzx_decode_match(main_element,
                                                     block_type,
                                                     end - window_pos,
@@ -793,7 +686,7 @@ lzx_decompress_block(int block_type, unsigned block_size,
                                                     tables,
                                                     queue,
                                                     istream);
-                       if (match_len < 0)
+                       if (unlikely(match_len < 0))
                                return match_len;
                        window_pos += match_len;
                }
@@ -801,12 +694,12 @@ lzx_decompress_block(int block_type, unsigned block_size,
        return 0;
 }
 
-/* API function documented in wimlib.h  */
-WIMLIBAPI int
-wimlib_lzx_decompress(const void *compressed_data, unsigned compressed_len,
-                     void *uncompressed_data, unsigned uncompressed_len)
+static int
+lzx_decompress(const void *compressed_data, size_t compressed_size,
+              void *uncompressed_data, size_t uncompressed_size,
+              void *_ctx)
 {
-       struct lzx_tables tables;
+       struct lzx_decompressor *ctx = _ctx;
        struct input_bitstream istream;
        struct lzx_lru_queue queue;
        unsigned window_pos;
@@ -815,19 +708,24 @@ wimlib_lzx_decompress(const void *compressed_data, unsigned compressed_len,
        int ret;
        bool e8_preprocessing_done;
 
-       LZX_DEBUG("lzx_decompress (compressed_data = %p, compressed_len = %d, "
-                 "uncompressed_data = %p, uncompressed_len = %d).",
-                 compressed_data, compressed_len,
-                 uncompressed_data, uncompressed_len);
-
-       wimlib_assert(uncompressed_len <= 32768);
+       LZX_DEBUG("compressed_data = %p, compressed_size = %zu, "
+                 "uncompressed_data = %p, uncompressed_size = %zu, "
+                 "max_window_size=%u).",
+                 compressed_data, compressed_size,
+                 uncompressed_data, uncompressed_size,
+                 ctx->max_window_size);
+
+       if (uncompressed_size > ctx->max_window_size) {
+               LZX_DEBUG("Uncompressed size of %zu exceeds "
+                         "window size of %u!",
+                         uncompressed_size, ctx->max_window_size);
+               return -1;
+       }
 
-       memset(tables.maintree_lens, 0, sizeof(tables.maintree_lens));
-       memset(tables.lentree_lens, 0, sizeof(tables.lentree_lens));
-       queue.R0 = 1;
-       queue.R1 = 1;
-       queue.R2 = 1;
-       init_input_bitstream(&istream, compressed_data, compressed_len);
+       memset(ctx->tables.maintree_lens, 0, sizeof(ctx->tables.maintree_lens));
+       memset(ctx->tables.lentree_lens, 0, sizeof(ctx->tables.lentree_lens));
+       lzx_lru_queue_init(&queue);
+       init_input_bitstream(&istream, compressed_data, compressed_size);
 
        e8_preprocessing_done = false; /* Set to true if there may be 0xe8 bytes
                                          in the uncompressed data. */
@@ -838,22 +736,23 @@ wimlib_lzx_decompress(const void *compressed_data, unsigned compressed_len,
         * blocks.  */
 
        for (window_pos = 0;
-            window_pos < uncompressed_len;
+            window_pos < uncompressed_size;
             window_pos += block_size)
        {
                LZX_DEBUG("Reading block header.");
-               ret = lzx_read_block_header(&istream, &block_size,
-                                           &block_type, &tables, &queue);
+               ret = lzx_read_block_header(&istream, ctx->num_main_syms,
+                                           ctx->max_window_size, &block_size,
+                                           &block_type, &ctx->tables, &queue);
                if (ret)
                        return ret;
 
                LZX_DEBUG("block_size = %u, window_pos = %u",
                          block_size, window_pos);
 
-               if (block_size > uncompressed_len - window_pos) {
+               if (block_size > uncompressed_size - window_pos) {
                        LZX_DEBUG("Expected a block size of at "
-                                 "most %u bytes (found %u bytes)",
-                                 uncompressed_len - window_pos, block_size);
+                                 "most %zu bytes (found %u bytes)",
+                                 uncompressed_size - window_pos, block_size);
                        return -1;
                }
 
@@ -868,12 +767,13 @@ wimlib_lzx_decompress(const void *compressed_data, unsigned compressed_len,
                                                   block_size,
                                                   uncompressed_data,
                                                   window_pos,
-                                                  &tables,
+                                                  &ctx->tables,
                                                   &queue,
                                                   &istream);
                        if (ret)
                                return ret;
-                       if (tables.maintree_lens[0xe8] != 0)
+
+                       if (ctx->tables.maintree_lens[0xe8] != 0)
                                e8_preprocessing_done = true;
                        break;
                case LZX_BLOCKTYPE_UNCOMPRESSED:
@@ -900,6 +800,42 @@ wimlib_lzx_decompress(const void *compressed_data, unsigned compressed_len,
                }
        }
        if (e8_preprocessing_done)
-               undo_call_insn_preprocessing(uncompressed_data, uncompressed_len);
+               lzx_undo_e8_preprocessing(uncompressed_data, uncompressed_size);
+       return 0;
+}
+
+static void
+lzx_free_decompressor(void *_ctx)
+{
+       struct lzx_decompressor *ctx = _ctx;
+
+       ALIGNED_FREE(ctx);
+}
+
+static int
+lzx_create_decompressor(size_t max_window_size,
+                       const struct wimlib_decompressor_params_header *params,
+                       void **ctx_ret)
+{
+       struct lzx_decompressor *ctx;
+
+       if (!lzx_window_size_valid(max_window_size))
+               return WIMLIB_ERR_INVALID_PARAM;
+
+       ctx = ALIGNED_MALLOC(sizeof(struct lzx_decompressor),
+                            DECODE_TABLE_ALIGNMENT);
+       if (ctx == NULL)
+               return WIMLIB_ERR_NOMEM;
+
+       ctx->max_window_size = max_window_size;
+       ctx->num_main_syms = lzx_get_num_main_syms(max_window_size);
+
+       *ctx_ret = ctx;
        return 0;
 }
+
+const struct decompressor_ops lzx_decompressor_ops = {
+       .create_decompressor = lzx_create_decompressor,
+       .decompress          = lzx_decompress,
+       .free_decompressor   = lzx_free_decompressor,
+};