random small shrinkage and elimination of statics
[oweals/busybox.git] / archival / libunarchive / decompress_unzip.c
index 0a199dd9f0597ba16c824715ec25f3c4e938ed57..331fe34d844eb2bc1d9c862bfede989ba68834bb 100644 (file)
  * to free leaked bytebuffer memory (used in unzip.c), and some minor style
  * guide cleanups by Ed Clark
  *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
- *
  * gzip (GNU zip) -- compress files with zip algorithm and 'compress' interface
  * Copyright (C) 1992-1993 Jean-loup Gailly
  * The unzip code was written and put in the public domain by Mark Adler.
  * written by Spencer Thomas, Joe Orost, James Woods, Jim McKie, Steve Davies,
  * Ken Turkowski, Dave Mack and Peter Jannesen.
  *
- * See the license_msg below and the file COPYING for the software license.
  * See the file algorithm.doc for the compression algorithms and file formats.
+ *
+ * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#if 0
-static char *license_msg[] = {
-       "   Copyright (C) 1992-1993 Jean-loup Gailly",
-       "   This program is free software; you can redistribute it and/or modify",
-       "   it under the terms of the GNU General Public License as published by",
-       "   the Free Software Foundation; either version 2, or (at your option)",
-       "   any later version.",
-       "",
-       "   This program is distributed in the hope that it will be useful,",
-       "   but WITHOUT ANY WARRANTY; without even the implied warranty of",
-       "   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the",
-       "   GNU General Public License for more details.",
-       "",
-       "   You should have received a copy of the GNU General Public License",
-       "   along with this program; if not, write to the Free Software",
-       "   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.",
-       0
-};
-#endif
-
-#include <sys/types.h>
-#include <sys/wait.h>
-#include <signal.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
-#include <fcntl.h>
-#include "config.h"
-#include "busybox.h"
+#include "libbb.h"
 #include "unarchive.h"
 
 typedef struct huft_s {
@@ -87,31 +45,142 @@ typedef struct huft_s {
        } v;
 } huft_t;
 
-static int gunzip_src_fd;
-unsigned int gunzip_bytes_out; /* number of output bytes */
-static unsigned int gunzip_outbuf_count;       /* bytes in output buffer */
-
-/* gunzip_window size--must be a power of two, and
*  at least 32K for zip's deflate method */
-static const int gunzip_wsize = 0x8000;
-static unsigned char *gunzip_window;
+enum {
+       /* gunzip_window size--must be a power of two, and
+        *  at least 32K for zip's deflate method */
+       GUNZIP_WSIZE = 0x8000,
+       /* If BMAX needs to be larger than 16, then h and x[] should be ulg. */
      BMAX = 16,      /* maximum bit length of any code (16 for explode) */
+       N_MAX = 288,    /* maximum number of codes in any set */
+};
 
-static unsigned int *gunzip_crc_table;
-unsigned int gunzip_crc;
 
-/* If BMAX needs to be larger than 16, then h and x[] should be ulg. */
-#define BMAX 16        /* maximum bit length of any code (16 for explode) */
-#define N_MAX 288      /* maximum number of codes in any set */
+/* This is somewhat complex-looking arrangement, but it allows
+ * to place decompressor state either in bss or in
+ * malloc'ed space simply by changing #defines below.
+ * Sizes on i386:
+ * text    data     bss     dec     hex
+ * 5256       0     108    5364    14f4 - bss
+ * 4915       0       0    4915    1333 - malloc
+ */
+#define STATE_IN_BSS 0
+#define STATE_IN_MALLOC 1
+
+
+typedef struct state_t {
+       off_t gunzip_bytes_out; /* number of output bytes */
+       uint32_t gunzip_crc;
+
+       int gunzip_src_fd;
+       unsigned gunzip_outbuf_count; /* bytes in output buffer */
+
+       unsigned char *gunzip_window;
+
+       uint32_t *gunzip_crc_table;
+
+       /* bitbuffer */
+       unsigned gunzip_bb; /* bit buffer */
+       unsigned char gunzip_bk; /* bits in bit buffer */
+
+       /* These control the size of the STATE()bytebuffer */
+       unsigned bytebuffer_max;
+       unsigned char *bytebuffer;
+       unsigned bytebuffer_offset;
+       unsigned bytebuffer_size;
+
+       /* private data of inflate_codes() */
+       unsigned inflate_codes_ml; /* masks for bl and bd bits */
+       unsigned inflate_codes_md; /* masks for bl and bd bits */
+       unsigned inflate_codes_bb; /* bit buffer */
+       unsigned inflate_codes_k; /* number of bits in bit buffer */
+       unsigned inflate_codes_w; /* current gunzip_window position */
+       huft_t *inflate_codes_tl;
+       huft_t *inflate_codes_td;
+       unsigned inflate_codes_bl;
+       unsigned inflate_codes_bd;
+       unsigned inflate_codes_nn; /* length and index for copy */
+       unsigned inflate_codes_dd;
+       smallint resume_copy;
+
+       /* private data of inflate_get_next_window() */
+       smallint method; /* Method == -1 for stored, -2 for codes */
+       smallint need_another_block;
+       smallint end_reached;
+
+       /* private data of inflate_stored() */
+       unsigned inflate_stored_n;
+       unsigned inflate_stored_b;
+       unsigned inflate_stored_k;
+       unsigned inflate_stored_w;
+} state_t;
+#define gunzip_bytes_out    (S()gunzip_bytes_out   )
+#define gunzip_crc          (S()gunzip_crc         )
+#define gunzip_src_fd       (S()gunzip_src_fd      )
+#define gunzip_outbuf_count (S()gunzip_outbuf_count)
+#define gunzip_window       (S()gunzip_window      )
+#define gunzip_crc_table    (S()gunzip_crc_table   )
+#define gunzip_bb           (S()gunzip_bb          )
+#define gunzip_bk           (S()gunzip_bk          )
+#define bytebuffer_max      (S()bytebuffer_max     )
+#define bytebuffer          (S()bytebuffer         )
+#define bytebuffer_offset   (S()bytebuffer_offset  )
+#define bytebuffer_size     (S()bytebuffer_size    )
+#define inflate_codes_ml    (S()inflate_codes_ml   )
+#define inflate_codes_md    (S()inflate_codes_md   )
+#define inflate_codes_bb    (S()inflate_codes_bb   )
+#define inflate_codes_k     (S()inflate_codes_k    )
+#define inflate_codes_w     (S()inflate_codes_w    )
+#define inflate_codes_tl    (S()inflate_codes_tl   )
+#define inflate_codes_td    (S()inflate_codes_td   )
+#define inflate_codes_bl    (S()inflate_codes_bl   )
+#define inflate_codes_bd    (S()inflate_codes_bd   )
+#define inflate_codes_nn    (S()inflate_codes_nn   )
+#define inflate_codes_dd    (S()inflate_codes_dd   )
+#define resume_copy         (S()resume_copy        )
+#define method              (S()method             )
+#define need_another_block  (S()need_another_block )
+#define end_reached         (S()end_reached        )
+#define inflate_stored_n    (S()inflate_stored_n   )
+#define inflate_stored_b    (S()inflate_stored_b   )
+#define inflate_stored_k    (S()inflate_stored_k   )
+#define inflate_stored_w    (S()inflate_stored_w   )
+#define INIT_STATE ({ bytebuffer_size = 0; method = -1; need_another_block = 1; })
+
+
+/* This is generic part */
+#if STATE_IN_BSS /* Use global data segment */
+#define DECLARE_STATE /*nothing*/
+#define ALLOC_STATE (init_state())
+#define DEALLOC_STATE ((void)0)
+#define S() state.
+#define PASS_STATE /*nothing*/
+#define PASS_STATE_ONLY /*nothing*/
+#define STATE_PARAM /*nothing*/
+#define STATE_PARAM_ONLY void
+static state_t state;
+static void init_state(void)
+{
+       INIT_STATE;
+}
+#endif
 
-/* bitbuffer */
-static unsigned int gunzip_bb; /* bit buffer */
-static unsigned char gunzip_bk;        /* bits in bit buffer */
+#if STATE_IN_MALLOC /* Use malloc space */
+#define DECLARE_STATE state_t *state
+#define ALLOC_STATE (state = alloc_state())
+#define DEALLOC_STATE free(state)
+#define S() state->
+#define PASS_STATE state,
+#define PASS_STATE_ONLY state
+#define STATE_PARAM state_t *state,
+#define STATE_PARAM_ONLY state_t *state
+static state_t* alloc_state(void)
+{
+       state_t* state = xzalloc(sizeof(*state));
+       INIT_STATE;
+       return state;
+}
+#endif
 
-/* These control the size of the bytebuffer */
-static unsigned int bytebuffer_max = 0x8000;
-static unsigned char *bytebuffer = NULL;
-static unsigned int bytebuffer_offset = 0;
-static unsigned int bytebuffer_size = 0;
 
 static const unsigned short mask_bits[] = {
        0x0000, 0x0001, 0x0003, 0x0007, 0x000f, 0x001f, 0x003f, 0x007f, 0x00ff,
@@ -129,7 +198,7 @@ static const unsigned short cplens[] = {
 static const unsigned char cplext[] = {
        0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 2, 2, 2, 2, 3, 3, 3, 3, 4, 4, 4, 4, 5,
        5, 5, 5, 0, 99, 99
-};                                             /* 99==invalid */
+}; /* 99 == invalid */
 
 /* Copy offsets for distance codes 0..29 */
 static const unsigned short cpdist[] = {
@@ -149,53 +218,25 @@ static const unsigned char border[] = {
        16, 17, 18, 0, 8, 7, 9, 6, 10, 5, 11, 4, 12, 3, 13, 2, 14, 1, 15
 };
 
-static unsigned int fill_bitbuffer(unsigned int bitbuffer, unsigned int *current, const unsigned int required)
+static unsigned fill_bitbuffer(STATE_PARAM unsigned bitbuffer, unsigned *current, const unsigned required)
 {
        while (*current < required) {
                if (bytebuffer_offset >= bytebuffer_size) {
                        /* Leave the first 4 bytes empty so we can always unwind the bitbuffer
                         * to the front of the bytebuffer, leave 4 bytes free at end of tail
                         * so we can easily top up buffer in check_trailer_gzip() */
-                       if (!(bytebuffer_size = bb_xread(gunzip_src_fd, &bytebuffer[4], bytebuffer_max - 8))) {
+                       bytebuffer_size = safe_read(gunzip_src_fd, &bytebuffer[4], bytebuffer_max - 8);
+                       if (1 > bytebuffer_size)
+//shouldn't we propagate error?
                                bb_error_msg_and_die("unexpected end of file");
-                       }
                        bytebuffer_size += 4;
                        bytebuffer_offset = 4;
                }
-               bitbuffer |= ((unsigned int) bytebuffer[bytebuffer_offset]) << *current;
+               bitbuffer |= ((unsigned) bytebuffer[bytebuffer_offset]) << *current;
                bytebuffer_offset++;
                *current += 8;
        }
-       return(bitbuffer);
-}
-
-static void make_gunzip_crc_table(void)
-{
-       const unsigned int poly = 0xedb88320;   /* polynomial exclusive-or pattern */
-       unsigned short i;       /* counter for all possible eight bit values */
-
-       /* initial shift register value */
-       gunzip_crc = 0xffffffffL;
-       gunzip_crc_table = (unsigned int *) xmalloc(256 * sizeof(unsigned int));
-
-       /* Compute and print table of CRC's, five per line */
-       for (i = 0; i < 256; i++) {
-               unsigned int table_entry;       /* crc shift register */
-               unsigned char k;        /* byte being shifted into crc apparatus */
-
-               table_entry = i;
-               /* The idea to initialize the register with the byte instead of
-                  * zero was stolen from Haruhiko Okumura's ar002
-                */
-               for (k = 8; k; k--) {
-                       if (table_entry & 1) {
-                               table_entry = (table_entry >> 1) ^ poly;
-                       } else {
-                               table_entry >>= 1;
-                       }
-               }
-               gunzip_crc_table[i] = table_entry;
-       }
+       return bitbuffer;
 }
 
 /*
@@ -204,19 +245,16 @@ static void make_gunzip_crc_table(void)
  * each table.
  * t: table to free
  */
-static int huft_free(huft_t * t)
+static void huft_free(huft_t * p)
 {
-       huft_t *p;
        huft_t *q;
 
        /* Go through linked list, freeing from the malloced (t[-1]) address. */
-       p = t;
-       while (p != (huft_t *) NULL) {
+       while (p) {
                q = (--p)->v.t;
-               free((char *) p);
+               free(p);
                p = q;
        }
-       return 0;
 }
 
 /* Given a list of code lengths and a maximum table size, make a set of
@@ -233,26 +271,26 @@ static int huft_free(huft_t * t)
  * t:  result: starting table
  * m:  maximum lookup bits, returns actual
  */
-int huft_build(unsigned int *b, const unsigned int n,
-                          const unsigned int s, const unsigned short *d,
-                          const unsigned char *e, huft_t ** t, int *m)
+static int huft_build(unsigned *b, const unsigned n,
+                          const unsigned s, const unsigned short *d,
+                          const unsigned char *e, huft_t ** t, unsigned *m)
 {
        unsigned a;                             /* counter for codes of length k */
        unsigned c[BMAX + 1];   /* bit length count table */
        unsigned eob_len;               /* length of end-of-block code (value 256) */
        unsigned f;                             /* i repeats in table every f entries */
        int g;                                  /* maximum code length */
-       int h;                                  /* table level */
-       register unsigned i;    /* counter, current code */
-       register unsigned j;    /* counter */
-       register int k;                 /* number of bits in current code */
-       register unsigned *p;   /* pointer into c[], b[], or v[] */
-       register huft_t *q;             /* points to current table */
+       int htl;                                /* table level */
+       unsigned i;                             /* counter, current code */
+       unsigned j;                             /* counter */
+       int k;                                  /* number of bits in current code */
+       unsigned *p;                    /* pointer into c[], b[], or v[] */
+       huft_t *q;                              /* points to current table */
        huft_t r;                               /* table entry for structure assignment */
        huft_t *u[BMAX];                /* table stack */
        unsigned v[N_MAX];              /* values in order of bit length */
        int ws[BMAX+1];                 /* bits decoded stack */
-       register int w;                 /* bits decoded */
+       int w;                                  /* bits decoded */
        unsigned x[BMAX + 1];   /* bit offsets, then code stack */
        unsigned *xp;                   /* pointer into x */
        int y;                                  /* number of dummy codes added */
@@ -262,7 +300,7 @@ int huft_build(unsigned int *b, const unsigned int n,
        eob_len = n > 256 ? b[256] : BMAX;
 
        /* Generate counts for each bit length */
-       memset((void *)c, 0, sizeof(c));
+       memset(c, 0, sizeof(c));
        p = b;
        i = n;
        do {
@@ -270,9 +308,9 @@ int huft_build(unsigned int *b, const unsigned int n,
                p++; /* Can't combine with above line (Solaris bug) */
        } while (--i);
        if (c[0] == n) { /* null input--all zero length codes */
-               *t = (huft_t *) NULL;
+               *t = NULL;
                *m = 0;
-               return 0;
+               return 2;
        }
 
        /* Find minimum and maximum length, bound *m by those */
@@ -284,11 +322,13 @@ int huft_build(unsigned int *b, const unsigned int n,
 
        /* Adjust last length count to fill out codes, if needed */
        for (y = 1 << j; j < i; j++, y <<= 1) {
-               if ((y -= c[j]) < 0) {
+               y -= c[j];
+               if (y < 0) {
                        return 2; /* bad input: more codes than bits */
                }
        }
-       if ((y -= c[i]) < 0) {
+       y -= c[i];
+       if (y < 0) {
                return 2;
        }
        c[i] += y;
@@ -298,14 +338,16 @@ int huft_build(unsigned int *b, const unsigned int n,
        p = c + 1;
        xp = x + 2;
        while (--i) { /* note that i == g from above */
-               *xp++ = (j += *p++);
+               j += *p++;
+               *xp++ = j;
        }
 
        /* Make a table of values in order of bit lengths */
        p = b;
        i = 0;
        do {
-               if ((j = *p++) != 0) {
+               j = *p++;
+               if (j != 0) {
                        v[x[j]++] = i;
                }
        } while (++i < n);
@@ -313,10 +355,10 @@ int huft_build(unsigned int *b, const unsigned int n,
        /* Generate the Huffman codes and for each, make the table entries */
        x[0] = i = 0;                   /* first Huffman code is zero */
        p = v;                                  /* grab values in bit order */
-       h = -1;                                 /* no tables yet--level -1 */
+       htl = -1;                               /* no tables yet--level -1 */
        w = ws[0] = 0;                  /* bits decoded */
-       u[0] = (huft_t *) NULL; /* just to keep compilers happy */
-       q = (huft_t *) NULL;    /* ditto */
+       u[0] = NULL;    /* just to keep compilers happy */
+       q = NULL;       /* ditto */
        z = 0;                                  /* ditto */
 
        /* go through the bit lengths (k already is bits in shortest code) */
@@ -325,17 +367,21 @@ int huft_build(unsigned int *b, const unsigned int n,
                while (a--) {
                        /* here i is the Huffman code of length k bits for value *p */
                        /* make tables up to required level */
-                       while (k > ws[h + 1]) {
-                               w = ws[++h];
+                       while (k > ws[htl + 1]) {
+                               w = ws[++htl];
 
                                /* compute minimum size table less than or equal to *m bits */
-                               z = (z = g - w) > *m ? *m : z; /* upper limit on table size */
-                               if ((f = 1 << (j = k - w)) > a + 1) { /* try a k-w bit table */
+                               z = g - w;
+                               z = z > *m ? *m : z; /* upper limit on table size */
+                               j = k - w;
+                               f = 1 << j;
+                               if (f > a + 1) { /* try a k-w bit table */
                                        /* too few codes for k-w bit table */
                                        f -= a + 1; /* deduct codes from patterns left */
                                        xp = c + k;
                                        while (++j < z) { /* try smaller tables up to z bits */
-                                               if ((f <<= 1) <= *++xp) {
+                                               f <<= 1;
+                                               if (f <= *++xp) {
                                                        break; /* enough codes to use up j bits */
                                                }
                                                f -= *xp; /* else deduct codes from patterns */
@@ -343,22 +389,22 @@ int huft_build(unsigned int *b, const unsigned int n,
                                }
                                j = (w + j > eob_len && w < eob_len) ? eob_len - w : j; /* make EOB code end at table */
                                z = 1 << j;     /* table entries for j-bit table */
-                               ws[h+1] = w + j;        /* set bits decoded in stack */
+                               ws[htl+1] = w + j;      /* set bits decoded in stack */
 
                                /* allocate and link in new table */
-                               q = (huft_t *) xmalloc((z + 1) * sizeof(huft_t));
+                               q = xzalloc((z + 1) * sizeof(huft_t));
                                *t = q + 1;     /* link to list for huft_free() */
-                               *(t = &(q->v.t)) = NULL;
-                               u[h] = ++q;     /* table starts after link */
+                               t = &(q->v.t);
+                               u[htl] = ++q;   /* table starts after link */
 
                                /* connect to last table, if there is one */
-                               if (h) {
-                                       x[h] = i; /* save pattern for backing up */
-                                       r.b = (unsigned char) (w - ws[h - 1]); /* bits to dump before this table */
+                               if (htl) {
+                                       x[htl] = i; /* save pattern for backing up */
+                                       r.b = (unsigned char) (w - ws[htl - 1]); /* bits to dump before this table */
                                        r.e = (unsigned char) (16 + j); /* bits in this table */
                                        r.v.t = q; /* pointer to this table */
-                                       j = (i & ((1 << w) - 1)) >> ws[h - 1];
-                                       u[h - 1][j] = r; /* connect to last table */
+                                       j = (i & ((1 << w) - 1)) >> ws[htl - 1];
+                                       u[htl - 1][j] = r; /* connect to last table */
                                }
                        }
 
@@ -387,8 +433,8 @@ int huft_build(unsigned int *b, const unsigned int n,
                        i ^= j;
 
                        /* backup over finished tables */
-                       while ((i & ((1 << w) - 1)) != x[h]) {
-                               w = ws[--h];
+                       while ((i & ((1 << w) - 1)) != x[htl]) {
+                               w = ws[--htl];
                        }
                }
        }
@@ -400,6 +446,7 @@ int huft_build(unsigned int *b, const unsigned int n,
        return y != 0 && g != 1;
 }
 
+
 /*
  * inflate (decompress) the codes in a deflated (compressed) block.
  * Return an error code or zero if it all goes ok.
@@ -407,126 +454,141 @@ int huft_build(unsigned int *b, const unsigned int n,
  * tl, td: literal/length and distance decoder tables
  * bl, bd: number of bits decoded by tl[] and td[]
  */
-static int inflate_codes(huft_t * my_tl, huft_t * my_td, const unsigned int my_bl, const unsigned int my_bd, int setup)
+/* called once from inflate_block */
+
+/* map formerly local static variables to globals */
+#define ml inflate_codes_ml
+#define md inflate_codes_md
+#define bb inflate_codes_bb
+#define k  inflate_codes_k
+#define w  inflate_codes_w
+#define tl inflate_codes_tl
+#define td inflate_codes_td
+#define bl inflate_codes_bl
+#define bd inflate_codes_bd
+#define nn inflate_codes_nn
+#define dd inflate_codes_dd
+static void inflate_codes_setup(STATE_PARAM huft_t * my_tl, huft_t * my_td, const unsigned my_bl, const unsigned my_bd)
 {
-       static unsigned int e;  /* table entry flag/number of extra bits */
-       static unsigned int n, d;       /* length and index for copy */
-       static unsigned int w;  /* current gunzip_window position */
-       static huft_t *t;                       /* pointer to table entry */
-       static unsigned int ml, md;     /* masks for bl and bd bits */
-       static unsigned int b;  /* bit buffer */
-       static unsigned int k;                  /* number of bits in bit buffer */
-       static huft_t *tl, *td;
-       static unsigned int bl, bd;
-       static int resumeCopy = 0;
-
-       if (setup) { // 1st time we are called, copy in variables
-               tl = my_tl;
-               td = my_td;
-               bl = my_bl;
-               bd = my_bd;
-               /* make local copies of globals */
-               b = gunzip_bb;                          /* initialize bit buffer */
-               k = gunzip_bk;
-               w = gunzip_outbuf_count;                        /* initialize gunzip_window position */
-
-               /* inflate the coded data */
-               ml = mask_bits[bl];     /* precompute masks for speed */
-               md = mask_bits[bd];
-               return 0; // Don't actually do anything the first time
-       }
+       tl = my_tl;
+       td = my_td;
+       bl = my_bl;
+       bd = my_bd;
+       /* make local copies of globals */
+       bb = gunzip_bb;                 /* initialize bit buffer */
+       k = gunzip_bk;
+       w = gunzip_outbuf_count;        /* initialize gunzip_window position */
+       /* inflate the coded data */
+       ml = mask_bits[bl];             /* precompute masks for speed */
+       md = mask_bits[bd];
+}
+/* called once from inflate_get_next_window */
+static int inflate_codes(STATE_PARAM_ONLY)
+{
+       unsigned e;     /* table entry flag/number of extra bits */
+       huft_t *t;      /* pointer to table entry */
 
-       if (resumeCopy) goto do_copy;
+       if (resume_copy) goto do_copy;
 
        while (1) {                     /* do until end of block */
-               b = fill_bitbuffer(b, &k, bl);
-               if ((e = (t = tl + ((unsigned) b & ml))->e) > 16)
+               bb = fill_bitbuffer(PASS_STATE bb, &k, bl);
+               t = tl + ((unsigned) bb & ml);
+               e = t->e;
+               if (e > 16)
                        do {
                                if (e == 99) {
-                                       bb_error_msg_and_die("inflate_codes error 1");;
+//shouldn't we propagate error?
+                                       bb_error_msg_and_die("inflate_codes error 1");
                                }
-                               b >>= t->b;
+                               bb >>= t->b;
                                k -= t->b;
                                e -= 16;
-                               b = fill_bitbuffer(b, &k, e);
-                       } while ((e =
-                                         (t = t->v.t + ((unsigned) b & mask_bits[e]))->e) > 16);
-               b >>= t->b;
+                               bb = fill_bitbuffer(PASS_STATE bb, &k, e);
+                               t = t->v.t + ((unsigned) bb & mask_bits[e]);
+                               e = t->e;
+                       } while (e > 16);
+               bb >>= t->b;
                k -= t->b;
                if (e == 16) {  /* then it's a literal */
                        gunzip_window[w++] = (unsigned char) t->v.n;
-                       if (w == gunzip_wsize) {
-                               gunzip_outbuf_count = (w);
+                       if (w == GUNZIP_WSIZE) {
+                               gunzip_outbuf_count = w;
                                //flush_gunzip_window();
                                w = 0;
                                return 1; // We have a block to read
                        }
                } else {                /* it's an EOB or a length */
-
                        /* exit if end of block */
                        if (e == 15) {
                                break;
                        }
 
                        /* get length of block to copy */
-                       b = fill_bitbuffer(b, &k, e);
-                       n = t->v.n + ((unsigned) b & mask_bits[e]);
-                       b >>= e;
+                       bb = fill_bitbuffer(PASS_STATE bb, &k, e);
+                       nn = t->v.n + ((unsigned) bb & mask_bits[e]);
+                       bb >>= e;
                        k -= e;
 
                        /* decode distance of block to copy */
-                       b = fill_bitbuffer(b, &k, bd);
-                       if ((e = (t = td + ((unsigned) b & md))->e) > 16)
+                       bb = fill_bitbuffer(PASS_STATE bb, &k, bd);
+                       t = td + ((unsigned) bb & md);
+                       e = t->e;
+                       if (e > 16)
                                do {
                                        if (e == 99)
-                                               bb_error_msg_and_die("inflate_codes error 2");;
-                                       b >>= t->b;
+//shouldn't we propagate error?
+                                               bb_error_msg_and_die("inflate_codes error 2");
+                                       bb >>= t->b;
                                        k -= t->b;
                                        e -= 16;
-                                       b = fill_bitbuffer(b, &k, e);
-                               } while ((e =
-                                                 (t =
-                                                  t->v.t + ((unsigned) b & mask_bits[e]))->e) > 16);
-                       b >>= t->b;
+                                       bb = fill_bitbuffer(PASS_STATE bb, &k, e);
+                                       t = t->v.t + ((unsigned) bb & mask_bits[e]);
+                                       e = t->e;
+                               } while (e > 16);
+                       bb >>= t->b;
                        k -= t->b;
-                       b = fill_bitbuffer(b, &k, e);
-                       d = w - t->v.n - ((unsigned) b & mask_bits[e]);
-                       b >>= e;
+                       bb = fill_bitbuffer(PASS_STATE bb, &k, e);
+                       dd = w - t->v.n - ((unsigned) bb & mask_bits[e]);
+                       bb >>= e;
                        k -= e;
 
                        /* do the copy */
-do_copy:               do {
-                               n -= (e =
-                                         (e =
-                                          gunzip_wsize - ((d &= gunzip_wsize - 1) > w ? d : w)) > n ? n : e);
-                          /* copy to new buffer to prevent possible overwrite */
-                               if (w - d >= e) {       /* (this test assumes unsigned comparison) */
-                                       memcpy(gunzip_window + w, gunzip_window + d, e);
+ do_copy:
+                       do {
+                               /* Was: nn -= (e = (e = GUNZIP_WSIZE - ((dd &= GUNZIP_WSIZE - 1) > w ? dd : w)) > nn ? nn : e); */
+                               /* Who wrote THAT?? rewritten as: */
+                               dd &= GUNZIP_WSIZE - 1;
+                               e = GUNZIP_WSIZE - (dd > w ? dd : w);
+                               if (e > nn) e = nn;
+                               nn -= e;
+
+                               /* copy to new buffer to prevent possible overwrite */
+                               if (w - dd >= e) {      /* (this test assumes unsigned comparison) */
+                                       memcpy(gunzip_window + w, gunzip_window + dd, e);
                                        w += e;
-                                       d += e;
+                                       dd += e;
                                } else {
-                                  /* do it slow to avoid memcpy() overlap */
-                                  /* !NOMEMCPY */
+                                       /* do it slow to avoid memcpy() overlap */
+                                       /* !NOMEMCPY */
                                        do {
-                                               gunzip_window[w++] = gunzip_window[d++];
+                                               gunzip_window[w++] = gunzip_window[dd++];
                                        } while (--e);
                                }
-                               if (w == gunzip_wsize) {
-                                       gunzip_outbuf_count = (w);
-                                       if (n) resumeCopy = 1;
-                                       else resumeCopy = 0;
+                               if (w == GUNZIP_WSIZE) {
+                                       gunzip_outbuf_count = w;
+                                       resume_copy = (nn != 0);
                                        //flush_gunzip_window();
                                        w = 0;
                                        return 1;
                                }
-                       } while (n);
-                       resumeCopy = 0;
+                       } while (nn);
+                       resume_copy = 0;
                }
        }
 
        /* restore the globals from the locals */
-       gunzip_outbuf_count = w;                        /* restore global gunzip_window pointer */
-       gunzip_bb = b;                          /* restore global bit buffer */
+       gunzip_outbuf_count = w;        /* restore global gunzip_window pointer */
+       gunzip_bb = bb;                 /* restore global bit buffer */
        gunzip_bk = k;
 
        /* normally just after call to inflate_codes, but save code by putting it here */
@@ -537,53 +599,68 @@ do_copy:          do {
        /* done */
        return 0;
 }
-
-static int inflate_stored(int my_n, int my_b_stored, int my_k_stored, int setup)
+#undef ml
+#undef md
+#undef bb
+#undef k
+#undef w
+#undef tl
+#undef td
+#undef bl
+#undef bd
+#undef nn
+#undef dd
+
+
+/* called once from inflate_block */
+static void inflate_stored_setup(STATE_PARAM int my_n, int my_b, int my_k)
+{
+       inflate_stored_n = my_n;
+       inflate_stored_b = my_b;
+       inflate_stored_k = my_k;
+       /* initialize gunzip_window position */
+       inflate_stored_w = gunzip_outbuf_count;
+}
+/* called once from inflate_get_next_window */
+static int inflate_stored(STATE_PARAM_ONLY)
 {
-       static int n, b_stored, k_stored, w;
-       if (setup) {
-               n = my_n;
-               b_stored = my_b_stored;
-               k_stored = my_k_stored;
-               w = gunzip_outbuf_count;                /* initialize gunzip_window position */
-               return 0; // Don't do anything first time
-       }
-
        /* read and output the compressed data */
-       while (n--) {
-               b_stored = fill_bitbuffer(b_stored, &k_stored, 8);
-               gunzip_window[w++] = (unsigned char) b_stored;
-               if (w == (unsigned int) gunzip_wsize) {
-                       gunzip_outbuf_count = (w);
+       while (inflate_stored_n--) {
+               inflate_stored_b = fill_bitbuffer(PASS_STATE inflate_stored_b, &inflate_stored_k, 8);
+               gunzip_window[inflate_stored_w++] = (unsigned char) inflate_stored_b;
+               if (inflate_stored_w == GUNZIP_WSIZE) {
+                       gunzip_outbuf_count = inflate_stored_w;
                        //flush_gunzip_window();
-                       w = 0;
-                       b_stored >>= 8;
-                       k_stored -= 8;
+                       inflate_stored_w = 0;
+                       inflate_stored_b >>= 8;
+                       inflate_stored_k -= 8;
                        return 1; // We have a block
                }
-               b_stored >>= 8;
-               k_stored -= 8;
+               inflate_stored_b >>= 8;
+               inflate_stored_k -= 8;
        }
 
        /* restore the globals from the locals */
-       gunzip_outbuf_count = w;                /* restore global gunzip_window pointer */
-       gunzip_bb = b_stored;   /* restore global bit buffer */
-       gunzip_bk = k_stored;
+       gunzip_outbuf_count = inflate_stored_w;         /* restore global gunzip_window pointer */
+       gunzip_bb = inflate_stored_b;   /* restore global bit buffer */
+       gunzip_bk = inflate_stored_k;
        return 0; // Finished
 }
 
+
 /*
  * decompress an inflated block
  * e: last block flag
  *
  * GLOBAL VARIABLES: bb, kk,
  */
- // Return values: -1 = inflate_stored, -2 = inflate_codes
-static int inflate_block(int *e)
+/* Return values: -1 = inflate_stored, -2 = inflate_codes */
+/* One callsite in inflate_get_next_window */
+static int inflate_block(STATE_PARAM smallint *e)
 {
        unsigned t;                     /* block type */
-       register unsigned int b;        /* bit buffer */
-       unsigned int k; /* number of bits in bit buffer */
+       unsigned b;     /* bit buffer */
+       unsigned k;     /* number of bits in bit buffer */
 
        /* make local bit buffer */
 
@@ -591,13 +668,13 @@ static int inflate_block(int *e)
        k = gunzip_bk;
 
        /* read in last block bit */
-       b = fill_bitbuffer(b, &k, 1);
-       *e = (int) b & 1;
+       b = fill_bitbuffer(PASS_STATE b, &k, 1);
+       *e = b & 1;
        b >>= 1;
        k -= 1;
 
        /* read in block type */
-       b = fill_bitbuffer(b, &k, 2);
+       b = fill_bitbuffer(PASS_STATE b, &k, 2);
        t = (unsigned) b & 3;
        b >>= 2;
        k -= 2;
@@ -610,9 +687,9 @@ static int inflate_block(int *e)
        switch (t) {
        case 0:                 /* Inflate stored */
        {
-               unsigned int n; /* number of bytes in block */
-               unsigned int b_stored;  /* bit buffer */
-               unsigned int k_stored;  /* number of bits in bit buffer */
+               unsigned n;     /* number of bytes in block */
+               unsigned b_stored;      /* bit buffer */
+               unsigned k_stored;      /* number of bits in bit buffer */
 
                /* make local copies of globals */
                b_stored = gunzip_bb;   /* initialize bit buffer */
@@ -624,33 +701,34 @@ static int inflate_block(int *e)
                k_stored -= n;
 
                /* get the length and its complement */
-               b_stored = fill_bitbuffer(b_stored, &k_stored, 16);
+               b_stored = fill_bitbuffer(PASS_STATE b_stored, &k_stored, 16);
                n = ((unsigned) b_stored & 0xffff);
                b_stored >>= 16;
                k_stored -= 16;
 
-               b_stored = fill_bitbuffer(b_stored, &k_stored, 16);
+               b_stored = fill_bitbuffer(PASS_STATE b_stored, &k_stored, 16);
                if (n != (unsigned) ((~b_stored) & 0xffff)) {
                        return 1;       /* error in compressed data */
                }
                b_stored >>= 16;
                k_stored -= 16;
 
-               inflate_stored(n, b_stored, k_stored, 1); // Setup inflate_stored
+               inflate_stored_setup(PASS_STATE n, b_stored, k_stored); // Setup inflate_stored
+
                return -1;
        }
-       case 1:                 /* Inflate fixed
-                                                  * decompress an inflated type 1 (fixed Huffman codes) block.  We should
-                                                  * either replace this with a custom decoder, or at least precompute the
-                                                  * Huffman tables.
-                                                */
+       case 1:
+       /* Inflate fixed
+        * decompress an inflated type 1 (fixed Huffman codes) block.  We should
+        * either replace this with a custom decoder, or at least precompute the
+        * Huffman tables. */
        {
                int i;                  /* temporary variable */
                huft_t *tl;             /* literal/length code table */
                huft_t *td;             /* distance code table */
-               unsigned int bl;                        /* lookup bits for tl */
-               unsigned int bd;                        /* lookup bits for td */
-               unsigned int l[288];    /* length list for huft_build */
+               unsigned bl;                    /* lookup bits for tl */
+               unsigned bd;                    /* lookup bits for td */
+               unsigned l[288];        /* length list for huft_build */
 
                /* set up literal table */
                for (i = 0; i < 144; i++) {
@@ -666,7 +744,8 @@ static int inflate_block(int *e)
                        l[i] = 8;
                }
                bl = 7;
-               if ((i = huft_build(l, 288, 257, cplens, cplext, &tl, &bl)) != 0) {
+               i = huft_build(l, 288, 257, cplens, cplext, &tl, &bl);
+               if (i != 0) {
                        return i;
                }
 
@@ -675,13 +754,14 @@ static int inflate_block(int *e)
                        l[i] = 5;
                }
                bd = 5;
-               if ((i = huft_build(l, 30, 0, cpdist, cpdext, &td, &bd)) > 1) {
+               i = huft_build(l, 30, 0, cpdist, cpdext, &td, &bd);
+               if (i > 1) {
                        huft_free(tl);
                        return i;
                }
 
                /* decompress until an end-of-block code */
-               inflate_codes(tl, td, bl, bd, 1); // Setup inflate_codes
+               inflate_codes_setup(PASS_STATE tl, td, bl, bd); // Setup inflate_codes
 
                /* huft_free code moved into inflate_codes */
 
@@ -694,38 +774,38 @@ static int inflate_block(int *e)
 
                huft_t *tl;             /* literal/length code table */
                huft_t *td;             /* distance code table */
-               unsigned int i;                 /* temporary variables */
-               unsigned int j;
-               unsigned int l;         /* last length */
-               unsigned int m;         /* mask for bit lengths table */
-               unsigned int n;         /* number of lengths to get */
-               unsigned int bl;                        /* lookup bits for tl */
-               unsigned int bd;                        /* lookup bits for td */
-               unsigned int nb;        /* number of bit length codes */
-               unsigned int nl;        /* number of literal/length codes */
-               unsigned int nd;        /* number of distance codes */
-
-               unsigned int ll[286 + 30];      /* literal/length and distance code lengths */
-               unsigned int b_dynamic; /* bit buffer */
-               unsigned int k_dynamic; /* number of bits in bit buffer */
+               unsigned i;                     /* temporary variables */
+               unsigned j;
+               unsigned l;             /* last length */
+               unsigned m;             /* mask for bit lengths table */
+               unsigned n;             /* number of lengths to get */
+               unsigned bl;                    /* lookup bits for tl */
+               unsigned bd;                    /* lookup bits for td */
+               unsigned nb;    /* number of bit length codes */
+               unsigned nl;    /* number of literal/length codes */
+               unsigned nd;    /* number of distance codes */
+
+               unsigned ll[286 + 30];  /* literal/length and distance code lengths */
+               unsigned b_dynamic;     /* bit buffer */
+               unsigned k_dynamic;     /* number of bits in bit buffer */
 
                /* make local bit buffer */
                b_dynamic = gunzip_bb;
                k_dynamic = gunzip_bk;
 
                /* read in table lengths */
-               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 5);
-               nl = 257 + ((unsigned int) b_dynamic & 0x1f);   /* number of literal/length codes */
+               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 5);
+               nl = 257 + ((unsigned) b_dynamic & 0x1f);       /* number of literal/length codes */
 
                b_dynamic >>= 5;
                k_dynamic -= 5;
-               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 5);
-               nd = 1 + ((unsigned int) b_dynamic & 0x1f);     /* number of distance codes */
+               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 5);
+               nd = 1 + ((unsigned) b_dynamic & 0x1f); /* number of distance codes */
 
                b_dynamic >>= 5;
                k_dynamic -= 5;
-               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 4);
-               nb = 4 + ((unsigned int) b_dynamic & 0xf);      /* number of bit length codes */
+               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 4);
+               nb = 4 + ((unsigned) b_dynamic & 0xf);  /* number of bit length codes */
 
                b_dynamic >>= 4;
                k_dynamic -= 4;
@@ -735,8 +815,8 @@ static int inflate_block(int *e)
 
                /* read in bit-length-code lengths */
                for (j = 0; j < nb; j++) {
-                       b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 3);
-                       ll[border[j]] = (unsigned int) b_dynamic & 7;
+                       b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 3);
+                       ll[border[j]] = (unsigned) b_dynamic & 7;
                        b_dynamic >>= 3;
                        k_dynamic -= 3;
                }
@@ -758,31 +838,31 @@ static int inflate_block(int *e)
                n = nl + nd;
                m = mask_bits[bl];
                i = l = 0;
-               while ((unsigned int) i < n) {
-                       b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, (unsigned int)bl);
-                       j = (td = tl + ((unsigned int) b_dynamic & m))->b;
+               while ((unsigned) i < n) {
+                       b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, (unsigned)bl);
+                       j = (td = tl + ((unsigned) b_dynamic & m))->b;
                        b_dynamic >>= j;
                        k_dynamic -= j;
                        j = td->v.n;
                        if (j < 16) {   /* length of code in bits (0..15) */
                                ll[i++] = l = j;        /* save last length in l */
                        } else if (j == 16) {   /* repeat last length 3 to 6 times */
-                               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 2);
-                               j = 3 + ((unsigned int) b_dynamic & 3);
+                               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 2);
+                               j = 3 + ((unsigned) b_dynamic & 3);
                                b_dynamic >>= 2;
                                k_dynamic -= 2;
-                               if ((unsigned int) i + j > n) {
+                               if ((unsigned) i + j > n) {
                                        return 1;
                                }
                                while (j--) {
                                        ll[i++] = l;
                                }
                        } else if (j == 17) {   /* 3 to 10 zero length codes */
-                               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 3);
-                               j = 3 + ((unsigned int) b_dynamic & 7);
+                               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 3);
+                               j = 3 + ((unsigned) b_dynamic & 7);
                                b_dynamic >>= 3;
                                k_dynamic -= 3;
-                               if ((unsigned int) i + j > n) {
+                               if ((unsigned) i + j > n) {
                                        return 1;
                                }
                                while (j--) {
@@ -790,11 +870,11 @@ static int inflate_block(int *e)
                                }
                                l = 0;
                        } else {        /* j == 18: 11 to 138 zero length codes */
-                               b_dynamic = fill_bitbuffer(b_dynamic, &k_dynamic, 7);
-                               j = 11 + ((unsigned int) b_dynamic & 0x7f);
+                               b_dynamic = fill_bitbuffer(PASS_STATE b_dynamic, &k_dynamic, 7);
+                               j = 11 + ((unsigned) b_dynamic & 0x7f);
                                b_dynamic >>= 7;
                                k_dynamic -= 7;
-                               if ((unsigned int) i + j > n) {
+                               if ((unsigned) i + j > n) {
                                        return 1;
                                }
                                while (j--) {
@@ -814,26 +894,30 @@ static int inflate_block(int *e)
                /* build the decoding tables for literal/length and distance codes */
                bl = lbits;
 
-               if ((i = huft_build(ll, nl, 257, cplens, cplext, &tl, &bl)) != 0) {
+               i = huft_build(ll, nl, 257, cplens, cplext, &tl, &bl);
+               if (i != 0) {
                        if (i == 1) {
-                               bb_error_msg_and_die("Incomplete literal tree");
-                               huft_free(tl);
+//shouldn't we propagate error?
+                               bb_error_msg_and_die("incomplete literal tree");
+                               /* huft_free(tl); */
                        }
                        return i;       /* incomplete code set */
                }
 
                bd = dbits;
-               if ((i = huft_build(ll + nl, nd, 0, cpdist, cpdext, &td, &bd)) != 0) {
+               i = huft_build(ll + nl, nd, 0, cpdist, cpdext, &td, &bd);
+               if (i != 0) {
                        if (i == 1) {
+//shouldn't we propagate error?
                                bb_error_msg_and_die("incomplete distance tree");
-                               huft_free(td);
+                               /* huft_free(td); */
                        }
                        huft_free(tl);
                        return i;       /* incomplete code set */
                }
 
                /* decompress until an end-of-block code */
-               inflate_codes(tl, td, bl, bd, 1); // Setup inflate_codes
+               inflate_codes_setup(PASS_STATE tl, td, bl, bd); // Setup inflate_codes
 
                /* huft_free code moved into inflate_codes */
 
@@ -841,11 +925,13 @@ static int inflate_block(int *e)
        }
        default:
                /* bad block type */
-               bb_error_msg_and_die("bad block type %d\n", t);
+//shouldn't we propagate error?
+               bb_error_msg_and_die("bad block type %d", t);
        }
 }
 
-static void calculate_gunzip_crc(void)
+/* Two callsites, both in inflate_get_next_window */
+static void calculate_gunzip_crc(STATE_PARAM_ONLY)
 {
        int n;
        for (n = 0; n < gunzip_outbuf_count; n++) {
@@ -854,65 +940,57 @@ static void calculate_gunzip_crc(void)
        gunzip_bytes_out += gunzip_outbuf_count;
 }
 
-static int inflate_get_next_window(void)
+/* One callsite in inflate_unzip_internal */
+static int inflate_get_next_window(STATE_PARAM_ONLY)
 {
-       static int method = -1; // Method == -1 for stored, -2 for codes
-       static int e = 0;
-       static int needAnotherBlock = 1;
-
        gunzip_outbuf_count = 0;
 
-       while(1) {
+       while (1) {
                int ret;
 
-               if (needAnotherBlock) {
-                       if(e) {
-                               calculate_gunzip_crc();
-                               e = 0;
-                               needAnotherBlock = 1;
-                               return 0;
-                       } // Last block
-                       method = inflate_block(&e);
-                       needAnotherBlock = 0;
+               if (need_another_block) {
+                       if (end_reached) {
+                               calculate_gunzip_crc(PASS_STATE_ONLY);
+                               end_reached = 0;
+                               need_another_block = 1;
+                               return 0; /* Last block */
+                       }
+                       method = inflate_block(PASS_STATE &end_reached);
+                       need_another_block = 0;
                }
 
                switch (method) {
-                       case -1:        ret = inflate_stored(0,0,0,0);
-                                       break;
-                       case -2:        ret = inflate_codes(0,0,0,0,0);
-                                       break;
-                       default:        bb_error_msg_and_die("inflate error %d", method);
+               case -1:
+                       ret = inflate_stored(PASS_STATE_ONLY);
+                       break;
+               case -2:
+                       ret = inflate_codes(PASS_STATE_ONLY);
+                       break;
+               default:
+//shouldn't we propagate error?
+                       bb_error_msg_and_die("inflate error %d", method);
                }
 
                if (ret == 1) {
-                       calculate_gunzip_crc();
+                       calculate_gunzip_crc(PASS_STATE_ONLY);
                        return 1; // More data left
-               } else needAnotherBlock = 1; // End of that block
+               }
+               need_another_block = 1; // End of that block
        }
        /* Doesnt get here */
 }
 
-/* Initialise bytebuffer, be careful not to overfill the buffer */
-extern void inflate_init(unsigned int bufsize)
-{
-       /* Set the bytebuffer size, default is same as gunzip_wsize */
-       bytebuffer_max = bufsize + 8;
-       bytebuffer_offset = 4;
-       bytebuffer_size = 0;
-}
-
-extern void inflate_cleanup(void)
-{
-       free(bytebuffer);
-}
 
-extern int inflate_unzip(int in, int out)
+/* Called from inflate_gunzip() and inflate_unzip() */
+/* NB: bytebuffer is allocated here but freeing it is left to the caller! */
+static USE_DESKTOP(long long) int
+inflate_unzip_internal(STATE_PARAM int in, int out)
 {
+       USE_DESKTOP(long long) int n = 0;
        ssize_t nwrote;
-       typedef void (*sig_type) (int);
 
        /* Allocate all global buffers (for DYN_ALLOC option) */
-       gunzip_window = xmalloc(gunzip_wsize);
+       gunzip_window = xmalloc(GUNZIP_WSIZE);
        gunzip_outbuf_count = 0;
        gunzip_bytes_out = 0;
        gunzip_src_fd = in;
@@ -922,25 +1000,24 @@ extern int inflate_unzip(int in, int out)
        gunzip_bb = 0;
 
        /* Create the crc table */
-       make_gunzip_crc_table();
+       gunzip_crc_table = crc32_filltable(0);
+       gunzip_crc = ~0;
 
        /* Allocate space for buffer */
        bytebuffer = xmalloc(bytebuffer_max);
 
-       while(1) {
-               int ret = inflate_get_next_window();
-               nwrote = bb_full_write(out, gunzip_window, gunzip_outbuf_count);
-               if (nwrote == -1) {
+       while (1) {
+               int r = inflate_get_next_window(PASS_STATE_ONLY);
+               nwrote = full_write(out, gunzip_window, gunzip_outbuf_count);
+               if (nwrote != gunzip_outbuf_count) {
                        bb_perror_msg("write");
-                       return -1;
+                       n = -1;
+                       goto ret;
                }
-               if (ret == 0) break;
+               USE_DESKTOP(n += nwrote;)
+               if (r == 0) break;
        }
 
-       /* Cleanup */
-       free(gunzip_window);
-       free(gunzip_crc_table);
-
        /* Store unused bytes in a global buffer so calling applets can access it */
        if (gunzip_bk >= 8) {
                /* Undo too much lookahead. The next read will be byte aligned
@@ -950,20 +1027,54 @@ extern int inflate_unzip(int in, int out)
                gunzip_bb >>= 8;
                gunzip_bk -= 8;
        }
-       return 0;
+ ret:
+       /* Cleanup */
+       free(gunzip_window);
+       free(gunzip_crc_table);
+       return n;
+}
+
+
+USE_DESKTOP(long long) int
+inflate_unzip(inflate_unzip_result *res, unsigned bufsize, int in, int out)
+{
+       USE_DESKTOP(long long) int n;
+       DECLARE_STATE;
+
+       ALLOC_STATE;
+
+       bytebuffer_max = bufsize + 8;
+       bytebuffer_offset = 4;
+       n = inflate_unzip_internal(PASS_STATE in, out);
+
+       res->crc = gunzip_crc;
+       res->bytes_out = gunzip_bytes_out;
+       free(bytebuffer);
+       DEALLOC_STATE;
+       return n;
 }
 
-extern int inflate_gunzip(int in, int out)
+
+USE_DESKTOP(long long) int
+inflate_gunzip(int in, int out)
 {
-       unsigned int stored_crc = 0;
-       unsigned int count;
+       uint32_t stored_crc = 0;
+       unsigned count;
+       USE_DESKTOP(long long) int n;
+       DECLARE_STATE;
 
-       inflate_unzip(in, out);
+       ALLOC_STATE;
+
+       bytebuffer_max = 0x8000;
+       n = inflate_unzip_internal(PASS_STATE in, out);
+
+       if (n < 0) goto ret;
 
        /* top up the input buffer with the rest of the trailer */
        count = bytebuffer_size - bytebuffer_offset;
        if (count < 8) {
-               bb_xread_all(in, &bytebuffer[bytebuffer_size], 8 - count);
+               xread(in, &bytebuffer[bytebuffer_size], 8 - count);
+//shouldn't we propagate error?
                bytebuffer_size += 8 - count;
        }
        for (count = 0; count != 4; count++) {
@@ -972,16 +1083,22 @@ extern int inflate_gunzip(int in, int out)
        }
 
        /* Validate decompression - crc */
-       if (stored_crc != (gunzip_crc ^ 0xffffffffL)) {
+       if (stored_crc != (~gunzip_crc)) {
                bb_error_msg("crc error");
+               n = -1;
+               goto ret;
        }
 
        /* Validate decompression - size */
        if (gunzip_bytes_out !=
                (bytebuffer[bytebuffer_offset] | (bytebuffer[bytebuffer_offset+1] << 8) |
-               (bytebuffer[bytebuffer_offset+2] << 16) | (bytebuffer[bytebuffer_offset+3] << 24))) {
-               bb_error_msg("Incorrect length");
+               (bytebuffer[bytebuffer_offset+2] << 16) | (bytebuffer[bytebuffer_offset+3] << 24))
+       ) {
+               bb_error_msg("incorrect length");
+               n = -1;
        }
-
-       return 0;
+ ret:
+       free(bytebuffer);
+       DEALLOC_STATE;
+       return n;
 }