From acaaca839abada0d0718375ba03dcaeddda15d9e Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Sun, 30 May 2010 04:50:21 +0200 Subject: unxz: remove debugging. no code changes Signed-off-by: Denys Vlasenko --- archival/libunarchive/decompress_unxz.c | 1 - archival/libunarchive/unxz/xz.h | 4 ---- archival/libunarchive/unxz/xz_dec_lzma2.c | 4 +--- archival/libunarchive/unxz/xz_dec_stream.c | 35 +++++++++--------------------- 4 files changed, 11 insertions(+), 33 deletions(-) (limited to 'archival/libunarchive') diff --git a/archival/libunarchive/decompress_unxz.c b/archival/libunarchive/decompress_unxz.c index 9edc2461a..924a52513 100644 --- a/archival/libunarchive/decompress_unxz.c +++ b/archival/libunarchive/decompress_unxz.c @@ -12,7 +12,6 @@ #include "libbb.h" #include "unarchive.h" -//#define XZ_DEBUG_MSG(...) bb_error_msg(__VA_ARGS) #define XZ_REALLOC_DICT_BUF(ptr, size) xrealloc(ptr, size) #define XZ_FUNC FAST_FUNC #define XZ_EXTERN static diff --git a/archival/libunarchive/unxz/xz.h b/archival/libunarchive/unxz/xz.h index 82f16ee22..dbb9ba92d 100644 --- a/archival/libunarchive/unxz/xz.h +++ b/archival/libunarchive/unxz/xz.h @@ -19,10 +19,6 @@ # include #endif -#ifndef XZ_DEBUG_MSG -# define XZ_DEBUG_MSG(...) ((void)0) -#endif - /* In Linux, this is used to make extern functions static when needed. */ #ifndef XZ_EXTERN # define XZ_EXTERN extern diff --git a/archival/libunarchive/unxz/xz_dec_lzma2.c b/archival/libunarchive/unxz/xz_dec_lzma2.c index 890141b7c..c22dc5ba5 100644 --- a/archival/libunarchive/unxz/xz_dec_lzma2.c +++ b/archival/libunarchive/unxz/xz_dec_lzma2.c @@ -1117,10 +1117,8 @@ XZ_EXTERN enum xz_ret XZ_FUNC xz_dec_lzma2_reset( struct xz_dec_lzma2 *s, uint8_t props) { /* This limits dictionary size to 3 GiB to keep parsing simpler. */ - if (props > 39) { - XZ_DEBUG_MSG("props:%d", props); + if (props > 39) return XZ_OPTIONS_ERROR; - } s->dict.size = 2 + (props & 1); s->dict.size <<= (props >> 1) + 11; diff --git a/archival/libunarchive/unxz/xz_dec_stream.c b/archival/libunarchive/unxz/xz_dec_stream.c index e10c9413d..121c3b53a 100644 --- a/archival/libunarchive/unxz/xz_dec_stream.c +++ b/archival/libunarchive/unxz/xz_dec_stream.c @@ -365,16 +365,14 @@ static enum xz_ret XZ_FUNC dec_stream_header(struct xz_dec *s) /* * Decode the Stream Flags field. Of integrity checks, we support * only none (Check ID = 0) and CRC32 (Check ID = 1). + * We also accept CRC64 and SHA-256, but they will not be verified. */ if (s->temp.buf[HEADER_MAGIC_SIZE] != 0 - || (s->temp.buf[HEADER_MAGIC_SIZE + 1] > 1 - && s->temp.buf[HEADER_MAGIC_SIZE + 1] != 0x04 - && s->temp.buf[HEADER_MAGIC_SIZE + 1] != 0x0A - ) + || (s->temp.buf[HEADER_MAGIC_SIZE + 1] > 1 + && s->temp.buf[HEADER_MAGIC_SIZE + 1] != 0x04 /* CRC64 */ + && s->temp.buf[HEADER_MAGIC_SIZE + 1] != 0x0A /* SHA-256 */ + ) ) { - XZ_DEBUG_MSG("unsupported stream flags %x:%x", - s->temp.buf[HEADER_MAGIC_SIZE], - s->temp.buf[HEADER_MAGIC_SIZE+1]); return XZ_OPTIONS_ERROR; } @@ -435,10 +433,7 @@ static enum xz_ret XZ_FUNC dec_block_header(struct xz_dec *s) #else if (s->temp.buf[1] & 0x3F) #endif - { - XZ_DEBUG_MSG("s->temp.buf[1] & 0x3E/3F != 0"); return XZ_OPTIONS_ERROR; - } /* Compressed Size */ if (s->temp.buf[1] & 0x40) { @@ -466,10 +461,8 @@ static enum xz_ret XZ_FUNC dec_block_header(struct xz_dec *s) /* If there are two filters, the first one must be a BCJ filter. */ s->bcj_active = s->temp.buf[1] & 0x01; if (s->bcj_active) { - if (s->temp.size - s->temp.pos < 2) { - XZ_DEBUG_MSG("temp.size - temp.pos < 2"); + if (s->temp.size - s->temp.pos < 2) return XZ_OPTIONS_ERROR; - } ret = xz_dec_bcj_reset(s->bcj, s->temp.buf[s->temp.pos++]); if (ret != XZ_OK) @@ -479,10 +472,8 @@ static enum xz_ret XZ_FUNC dec_block_header(struct xz_dec *s) * We don't support custom start offset, * so Size of Properties must be zero. */ - if (s->temp.buf[s->temp.pos++] != 0x00) { - XZ_DEBUG_MSG("size of properties != 0"); + if (s->temp.buf[s->temp.pos++] != 0x00) return XZ_OPTIONS_ERROR; - } } #endif @@ -491,16 +482,12 @@ static enum xz_ret XZ_FUNC dec_block_header(struct xz_dec *s) return XZ_DATA_ERROR; /* Filter ID = LZMA2 */ - if (s->temp.buf[s->temp.pos++] != 0x21) { - XZ_DEBUG_MSG("filter ID != 0x21"); + if (s->temp.buf[s->temp.pos++] != 0x21) return XZ_OPTIONS_ERROR; - } /* Size of Properties = 1-byte Filter Properties */ - if (s->temp.buf[s->temp.pos++] != 0x01) { - XZ_DEBUG_MSG("size of properties != 1"); + if (s->temp.buf[s->temp.pos++] != 0x01) return XZ_OPTIONS_ERROR; - } /* Filter Properties contains LZMA2 dictionary size. */ if (s->temp.size - s->temp.pos < 1) @@ -512,10 +499,8 @@ static enum xz_ret XZ_FUNC dec_block_header(struct xz_dec *s) /* The rest must be Header Padding. */ while (s->temp.pos < s->temp.size) - if (s->temp.buf[s->temp.pos++] != 0x00) { - XZ_DEBUG_MSG("padding is not zero-filled"); + if (s->temp.buf[s->temp.pos++] != 0x00) return XZ_OPTIONS_ERROR; - } s->temp.pos = 0; s->block.compressed = 0; -- cgit v1.2.3