unxz: remove debugging. no code changes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/archival/libunarchive/decompress_unxz.c b/archival/libunarchive/decompress_unxz.c
index 9edc246..924a525 100644 (file)
--- 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 82f16ee..dbb9ba9 100644 (file)
--- a/archival/libunarchive/unxz/xz.h
+++ b/archival/libunarchive/unxz/xz.h
@@ -19,10 +19,6 @@
 #      include <stdint.h>
 #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 890141b..c22dc5b 100644 (file)
--- 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 e10c941..121c3b5 100644 (file)
--- 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;