aboutsummaryrefslogtreecommitdiff
path: root/src/liblzma/subblock
diff options
context:
space:
mode:
authorLasse Collin <lasse.collin@tukaani.org>2007-12-11 16:49:19 +0200
committerLasse Collin <lasse.collin@tukaani.org>2007-12-11 16:49:19 +0200
commit3e16d51dd645667b05ff826665b1fc353aa41cd9 (patch)
tree2ce26b05eacbec07557f3e4cbe0982dea05ec675 /src/liblzma/subblock
parentGet rid of no-NLS gnulib. I don't know how to get it (diff)
downloadxz-3e16d51dd645667b05ff826665b1fc353aa41cd9.tar.xz
Remove uncompressed size tracking from the filter encoders.
It's not strictly needed there, and just complicates the code. LZ encoder never even had this feature. The primary reason to have uncompressed size tracking in filter encoders was validating that the application doesn't give different amount of input that it had promised. A side effect was to validate internal workings of liblzma. Uncompressed size tracking is still present in the Block encoder. Maybe it should be added to LZMA_Alone and raw encoders too. It's simpler to have one coder just to validate the uncompressed size instead of having it in every filter.
Diffstat (limited to 'src/liblzma/subblock')
-rw-r--r--src/liblzma/subblock/subblock_encoder.c38
1 files changed, 6 insertions, 32 deletions
diff --git a/src/liblzma/subblock/subblock_encoder.c b/src/liblzma/subblock/subblock_encoder.c
index e1af4a45..96129d87 100644
--- a/src/liblzma/subblock/subblock_encoder.c
+++ b/src/liblzma/subblock/subblock_encoder.c
@@ -41,6 +41,7 @@ do { \
struct lzma_coder_s {
lzma_next_coder next;
bool next_finished;
+ bool use_eopm;
enum {
SEQ_FILL,
@@ -62,8 +63,6 @@ struct lzma_coder_s {
lzma_options_subblock *options;
- lzma_vli uncompressed_size;
-
size_t pos;
uint32_t tmp;
@@ -235,18 +234,6 @@ subblock_buffer(lzma_coder *coder, lzma_allocator *allocator,
size_t in_size, uint8_t *restrict out,
size_t *restrict out_pos, size_t out_size, lzma_action action)
{
- // Verify that there is a sane amount of input.
- if (coder->uncompressed_size != LZMA_VLI_VALUE_UNKNOWN) {
- const lzma_vli in_avail = in_size - *in_pos;
- if (action == LZMA_FINISH) {
- if (in_avail != coder->uncompressed_size)
- return LZMA_DATA_ERROR;
- } else {
- if (in_avail > coder->uncompressed_size)
- return LZMA_DATA_ERROR;
- }
- }
-
// Check if we need to do something special with the Subfilter.
if (coder->options != NULL && coder->options->allow_subfilters) {
switch (coder->options->subfilter_mode) {
@@ -304,18 +291,12 @@ subblock_buffer(lzma_coder *coder, lzma_allocator *allocator,
assert(coder->subfilter.subcoder.code == NULL);
// No Subfilter is enabled, just copy the data as is.
- // NOTE: uncompressed_size cannot overflow because we
- // have checked/ it in the beginning of this function.
- const size_t in_used = bufcpy(in, in_pos, in_size,
+ coder->alignment.in_pending += bufcpy(
+ in, in_pos, in_size,
coder->subblock.data,
&coder->subblock.size,
coder->subblock.limit);
- if (coder->uncompressed_size != LZMA_VLI_VALUE_UNKNOWN)
- coder->uncompressed_size -= in_used;
-
- coder->alignment.in_pending += in_used;
-
} else {
const size_t in_start = *in_pos;
lzma_ret ret;
@@ -351,11 +332,6 @@ subblock_buffer(lzma_coder *coder, lzma_allocator *allocator,
if (in_used > 0)
coder->subfilter.got_input = true;
- // NOTE: uncompressed_size cannot overflow because we
- // have checked it in the beginning of this function.
- if (coder->uncompressed_size != LZMA_VLI_VALUE_UNKNOWN)
- coder->uncompressed_size -= *in_pos - in_start;
-
coder->alignment.in_pending += in_used;
if (ret == LZMA_STREAM_END) {
@@ -509,14 +485,11 @@ subblock_buffer(lzma_coder *coder, lzma_allocator *allocator,
break;
}
- if (coder->uncompressed_size
- == LZMA_VLI_VALUE_UNKNOWN) {
+ if (coder->use_eopm) {
// NOTE: No need to use write_byte() here
// since we are finishing.
out[*out_pos] = 0x10;
++*out_pos;
- } else if (coder->uncompressed_size != 0) {
- return LZMA_DATA_ERROR;
}
return LZMA_STREAM_END;
@@ -782,7 +755,8 @@ lzma_subblock_encoder_init(lzma_next_coder *next, lzma_allocator *allocator,
next->coder->next_finished = false;
next->coder->sequence = SEQ_FILL;
next->coder->options = filters[0].options;
- next->coder->uncompressed_size = filters[0].uncompressed_size;
+ next->coder->use_eopm = filters[0].uncompressed_size
+ == LZMA_VLI_VALUE_UNKNOWN;
next->coder->pos = 0;
next->coder->alignment.in_pending = 0;