busybox: 1.11.0 fixes
authorPeter Korsgaard <jacmet@sunsite.dk>
Sat, 28 Jun 2008 20:53:52 +0000 (20:53 -0000)
committerPeter Korsgaard <jacmet@sunsite.dk>
Sat, 28 Jun 2008 20:53:52 +0000 (20:53 -0000)
package/busybox/busybox-1.11.0-bunzip2.patch [new file with mode: 0644]
package/busybox/busybox-1.11.0-ip.patch [new file with mode: 0644]
package/busybox/busybox-1.11.0-vi.patch [new file with mode: 0644]

diff --git a/package/busybox/busybox-1.11.0-bunzip2.patch b/package/busybox/busybox-1.11.0-bunzip2.patch
new file mode 100644 (file)
index 0000000..97f9fec
--- /dev/null
@@ -0,0 +1,503 @@
+--- busybox-1.11.0/archival/libunarchive/decompress_bunzip2.c  Wed Jun 25 14:51:26 2008
++++ busybox-1.11.0-bunzip2/archival/libunarchive/decompress_bunzip2.c  Sat Jun 28 20:04:31 2008
+@@ -66,7 +66,6 @@
+  *  | grep 'bd->' | sed 's/^.*bd->/bd->/' | sort | $PAGER
+  * and moved it (inbufBitCount) to offset 0.
+  */
+-
+ struct bunzip_data {
+       /* I/O tracking data (file handles, buffers, positions, etc.) */
+       unsigned inbufBitCount, inbufBits;
+@@ -102,11 +101,9 @@
+       /* If we need to get more data from the byte buffer, do so.  (Loop getting
+          one byte at a time to enforce endianness and avoid unaligned access.) */
+-
+       while ((int)(bd->inbufBitCount) < bits_wanted) {
+               /* If we need to read more data from file into byte buffer, do so */
+-
+               if (bd->inbufPos == bd->inbufCount) {
+                       /* if "no input fd" case: in_fd == -1, read fails, we jump */
+                       bd->inbufCount = read(bd->in_fd, bd->inbuf, IOBUF_SIZE);
+@@ -116,7 +113,6 @@
+               }
+               /* Avoid 32-bit overflow (dump bit buffer to top of output) */
+-
+               if (bd->inbufBitCount >= 24) {
+                       bits = bd->inbufBits & ((1 << bd->inbufBitCount) - 1);
+                       bits_wanted -= bd->inbufBitCount;
+@@ -125,13 +121,11 @@
+               }
+               /* Grab next 8 bits of input from buffer. */
+-
+               bd->inbufBits = (bd->inbufBits << 8) | bd->inbuf[bd->inbufPos++];
+               bd->inbufBitCount += 8;
+       }
+       /* Calculate result */
+-
+       bd->inbufBitCount -= bits_wanted;
+       bits |= (bd->inbufBits >> bd->inbufBitCount) & ((1 << bits_wanted) - 1);
+@@ -139,7 +133,6 @@
+ }
+ /* Unpacks the next block and sets up for the inverse burrows-wheeler step. */
+-
+ static int get_next_block(bunzip_data *bd)
+ {
+       struct group_data *hufGroup;
+@@ -153,13 +146,11 @@
+       selectors = bd->selectors;
+       /* Reset longjmp I/O error handling */
+-
+       i = setjmp(bd->jmpbuf);
+       if (i) return i;
+       /* Read in header signature and CRC, then validate signature.
+          (last block signature means CRC is for whole file, return now) */
+-
+       i = get_bits(bd, 24);
+       j = get_bits(bd, 24);
+       bd->headerCRC = get_bits(bd, 32);
+@@ -169,7 +160,6 @@
+       /* We can add support for blockRandomised if anybody complains.  There was
+          some code for this in busybox 1.0.0-pre3, but nobody ever noticed that
+          it didn't actually work. */
+-
+       if (get_bits(bd, 1)) return RETVAL_OBSOLETE_INPUT;
+       origPtr = get_bits(bd, 24);
+       if ((int)origPtr > dbufSize) return RETVAL_DATA_ERROR;
+@@ -179,7 +169,6 @@
+          symbols to deal with, and writes a sparse bitfield indicating which
+          values were present.  We make a translation table to convert the symbols
+          back to the corresponding bytes. */
+-
+       t = get_bits(bd, 16);
+       symTotal = 0;
+       for (i = 0; i < 16; i++) {
+@@ -192,7 +181,6 @@
+       }
+       /* How many different Huffman coding groups does this block use? */
+-
+       groupCount = get_bits(bd, 3);
+       if (groupCount < 2 || groupCount > MAX_GROUPS)
+               return RETVAL_DATA_ERROR;
+@@ -201,19 +189,16 @@
+          group.  Read in the group selector list, which is stored as MTF encoded
+          bit runs.  (MTF=Move To Front, as each value is used it's moved to the
+          start of the list.) */
+-
+       nSelectors = get_bits(bd, 15);
+       if (!nSelectors) return RETVAL_DATA_ERROR;
+       for (i = 0; i < groupCount; i++) mtfSymbol[i] = i;
+       for (i = 0; i < nSelectors; i++) {
+               /* Get next value */
+-
+               for (j = 0; get_bits(bd, 1); j++)
+                       if (j >= groupCount) return RETVAL_DATA_ERROR;
+               /* Decode MTF to get the next selector */
+-
+               uc = mtfSymbol[j];
+               for (;j;j--) mtfSymbol[j] = mtfSymbol[j-1];
+               mtfSymbol[0] = selectors[i] = uc;
+@@ -221,10 +206,11 @@
+       /* Read the Huffman coding tables for each group, which code for symTotal
+          literal symbols, plus two run symbols (RUNA, RUNB) */
+-
+       symCount = symTotal + 2;
+       for (j = 0; j < groupCount; j++) {
+-              unsigned char length[MAX_SYMBOLS], temp[MAX_HUFCODE_BITS+1];
++              unsigned char length[MAX_SYMBOLS];
++              /* 8 bits is ALMOST enough for temp[], see below */
++              unsigned temp[MAX_HUFCODE_BITS+1];
+               int minLen, maxLen, pp;
+               /* Read Huffman code lengths for each symbol.  They're stored in
+@@ -233,7 +219,6 @@
+                  (Subtracting 1 before the loop and then adding it back at the end is
+                  an optimization that makes the test inside the loop simpler: symbol
+                  length 0 becomes negative, so an unsigned inequality catches it.) */
+-
+               t = get_bits(bd, 5) - 1;
+               for (i = 0; i < symCount; i++) {
+                       for (;;) {
+@@ -243,7 +228,6 @@
+                               /* If first bit is 0, stop.  Else second bit indicates whether
+                                  to increment or decrement the value.  Optimization: grab 2
+                                  bits and unget the second if the first was 0. */
+-
+                               k = get_bits(bd, 2);
+                               if (k < 2) {
+                                       bd->inbufBitCount++;
+@@ -251,17 +235,14 @@
+                               }
+                               /* Add one if second bit 1, else subtract 1.  Avoids if/else */
+-
+                               t += (((k+1) & 2) - 1);
+                       }
+                       /* Correct for the initial -1, to get the final symbol length */
+-
+                       length[i] = t + 1;
+               }
+               /* Find largest and smallest lengths in this group */
+-
+               minLen = maxLen = length[0];
+               for (i = 1; i < symCount; i++) {
+                       if (length[i] > maxLen) maxLen = length[i];
+@@ -278,7 +259,6 @@
+                * number of bits can have.  This is how the Huffman codes can vary in
+                * length: each code with a value>limit[length] needs another bit.
+                */
+-
+               hufGroup = bd->groups + j;
+               hufGroup->minLen = minLen;
+               hufGroup->maxLen = maxLen;
+@@ -286,12 +266,10 @@
+               /* Note that minLen can't be smaller than 1, so we adjust the base
+                  and limit array pointers so we're not always wasting the first
+                  entry.  We do this again when using them (during symbol decoding).*/
+-
+               base = hufGroup->base - 1;
+               limit = hufGroup->limit - 1;
+               /* Calculate permute[].  Concurently, initialize temp[] and limit[]. */
+-
+               pp = 0;
+               for (i = minLen; i <= maxLen; i++) {
+                       temp[i] = limit[i] = 0;
+@@ -301,14 +279,14 @@
+               }
+               /* Count symbols coded for at each bit length */
+-
++              /* NB: in pathological cases, temp[8] can end ip being 256.
++               * That's why uint8_t is too small for temp[]. */
+               for (i = 0; i < symCount; i++) temp[length[i]]++;
+               /* Calculate limit[] (the largest symbol-coding value at each bit
+                * length, which is (previous limit<<1)+symbols at this level), and
+                * base[] (number of symbols to ignore at each bit length, which is
+                * limit minus the cumulative count of symbols coded for already). */
+-
+               pp = t = 0;
+               for (i = minLen; i < maxLen; i++) {
+                       pp += temp[i];
+@@ -319,7 +297,6 @@
+                          each level we're really only interested in the first few bits,
+                          so here we set all the trailing to-be-ignored bits to 1 so they
+                          don't affect the value>limit[length] comparison. */
+-
+                       limit[i] = (pp << (maxLen - i)) - 1;
+                       pp <<= 1;
+                       t += temp[i];
+@@ -335,7 +312,6 @@
+          and run length encoding, saving the result into dbuf[dbufCount++] = uc */
+       /* Initialize symbol occurrence counters and symbol Move To Front table */
+-
+       memset(byteCount, 0, sizeof(byteCount)); /* smaller, maybe slower? */
+       for (i = 0; i < 256; i++) {
+               //byteCount[i] = 0;
+@@ -347,8 +323,7 @@
+       runPos = dbufCount = selector = 0;
+       for (;;) {
+-              /* fetch next Huffman coding group from list. */
+-
++              /* Fetch next Huffman coding group from list. */
+               symCount = GROUP_SIZE - 1;
+               if (selector >= nSelectors) return RETVAL_DATA_ERROR;
+               hufGroup = bd->groups + selectors[selector++];
+@@ -367,7 +342,6 @@
+                  dry).  The following (up to got_huff_bits:) is equivalent to
+                  j = get_bits(bd, hufGroup->maxLen);
+                */
+-
+               while ((int)(bd->inbufBitCount) < hufGroup->maxLen) {
+                       if (bd->inbufPos == bd->inbufCount) {
+                               j = get_bits(bd, hufGroup->maxLen);
+@@ -382,13 +356,11 @@
+  got_huff_bits:
+               /* Figure how how many bits are in next symbol and unget extras */
+-
+               i = hufGroup->minLen;
+               while (j > limit[i]) ++i;
+               bd->inbufBitCount += (hufGroup->maxLen - i);
+               /* Huffman decode value to get nextSym (with bounds checking) */
+-
+               if (i > hufGroup->maxLen)
+                       return RETVAL_DATA_ERROR;
+               j = (j >> (hufGroup->maxLen - i)) - base[i];
+@@ -400,11 +372,9 @@
+                  byte, or a repeated run of the most recent literal byte.  First,
+                  check if nextSym indicates a repeated run, and if so loop collecting
+                  how many times to repeat the last literal. */
+-
+               if ((unsigned)nextSym <= SYMBOL_RUNB) { /* RUNA or RUNB */
+                       /* If this is the start of a new run, zero out counter */
+-
+                       if (!runPos) {
+                               runPos = 1;
+                               t = 0;
+@@ -417,7 +387,6 @@
+                          the basic or 0/1 method (except all bits 0, which would use no
+                          symbols, but a run of length 0 doesn't mean anything in this
+                          context).  Thus space is saved. */
+-
+                       t += (runPos << nextSym); /* +runPos if RUNA; +2*runPos if RUNB */
+                       if (runPos < dbufSize) runPos <<= 1;
+                       goto end_of_huffman_loop;
+@@ -427,7 +396,6 @@
+                  how many times to repeat the last literal, so append that many
+                  copies to our buffer of decoded symbols (dbuf) now.  (The last
+                  literal used is the one at the head of the mtfSymbol array.) */
+-
+               if (runPos) {
+                       runPos = 0;
+                       if (dbufCount + t >= dbufSize) return RETVAL_DATA_ERROR;
+@@ -438,7 +406,6 @@
+               }
+               /* Is this the terminating symbol? */
+-
+               if (nextSym > symTotal) break;
+               /* At this point, nextSym indicates a new literal character.  Subtract
+@@ -448,7 +415,6 @@
+                  first symbol in the mtf array, position 0, would have been handled
+                  as part of a run above.  Therefore 1 unused mtf position minus
+                  2 non-literal nextSym values equals -1.) */
+-
+               if (dbufCount >= dbufSize) return RETVAL_DATA_ERROR;
+               i = nextSym - 1;
+               uc = mtfSymbol[i];
+@@ -457,7 +423,6 @@
+                * small number of symbols, and are bound by 256 in any case, using
+                * memmove here would typically be bigger and slower due to function
+                * call overhead and other assorted setup costs. */
+-
+               do {
+                       mtfSymbol[i] = mtfSymbol[i-1];
+               } while (--i);
+@@ -465,13 +430,11 @@
+               uc = symToByte[uc];
+               /* We have our literal byte.  Save it into dbuf. */
+-
+               byteCount[uc]++;
+               dbuf[dbufCount++] = (unsigned)uc;
+               /* Skip group initialization if we're not done with this group.  Done
+                * this way to avoid compiler warning. */
+-
+  end_of_huffman_loop:
+               if (symCount--) goto continue_this_group;
+       }
+@@ -484,7 +447,6 @@
+        */
+       /* Turn byteCount into cumulative occurrence counts of 0 to n-1. */
+-
+       j = 0;
+       for (i = 0; i < 256; i++) {
+               k = j + byteCount[i];
+@@ -493,7 +455,6 @@
+       }
+       /* Figure out what order dbuf would be in if we sorted it. */
+-
+       for (i = 0; i < dbufCount; i++) {
+               uc = (unsigned char)(dbuf[i] & 0xff);
+               dbuf[byteCount[uc]] |= (i << 8);
+@@ -503,11 +464,10 @@
+       /* Decode first byte by hand to initialize "previous" byte.  Note that it
+          doesn't get output, and if the first three characters are identical
+          it doesn't qualify as a run (hence writeRunCountdown=5). */
+-
+       if (dbufCount) {
+               if ((int)origPtr >= dbufCount) return RETVAL_DATA_ERROR;
+               bd->writePos = dbuf[origPtr];
+-          bd->writeCurrent = (unsigned char)(bd->writePos & 0xff);
++              bd->writeCurrent = (unsigned char)(bd->writePos & 0xff);
+               bd->writePos >>= 8;
+               bd->writeRunCountdown = 5;
+       }
+@@ -522,7 +482,6 @@
+    error (all errors are negative numbers).  If out_fd!=-1, outbuf and len
+    are ignored, data is written to out_fd and return is RETVAL_OK or error.
+ */
+-
+ int read_bunzip(bunzip_data *bd, char *outbuf, int len)
+ {
+       const unsigned *dbuf;
+@@ -539,19 +498,15 @@
+       /* We will always have pending decoded data to write into the output
+          buffer unless this is the very first call (in which case we haven't
+          Huffman-decoded a block into the intermediate buffer yet). */
+-
+       if (bd->writeCopies) {
+               /* Inside the loop, writeCopies means extra copies (beyond 1) */
+-
+               --bd->writeCopies;
+               /* Loop outputting bytes */
+-
+               for (;;) {
+                       /* If the output buffer is full, snapshot state and return */
+-
+                       if (gotcount >= len) {
+                               bd->writePos = pos;
+                               bd->writeCurrent = current;
+@@ -560,13 +515,11 @@
+                       }
+                       /* Write next byte into output buffer, updating CRC */
+-
+                       outbuf[gotcount++] = current;
+                       bd->writeCRC = (bd->writeCRC << 8)
+-                                                ^ bd->crc32Table[(bd->writeCRC >> 24) ^ current];
++                              ^ bd->crc32Table[(bd->writeCRC >> 24) ^ current];
+                       /* Loop now if we're outputting multiple copies of this byte */
+-
+                       if (bd->writeCopies) {
+                               --bd->writeCopies;
+                               continue;
+@@ -582,35 +535,29 @@
+                       /* After 3 consecutive copies of the same byte, the 4th
+                        * is a repeat count.  We count down from 4 instead
+                        * of counting up because testing for non-zero is faster */
+-
+                       if (--bd->writeRunCountdown) {
+                               if (current != previous)
+                                       bd->writeRunCountdown = 4;
+                       } else {
+                               /* We have a repeated run, this byte indicates the count */
+-
+                               bd->writeCopies = current;
+                               current = previous;
+                               bd->writeRunCountdown = 5;
+                               /* Sometimes there are just 3 bytes (run length 0) */
+-
+                               if (!bd->writeCopies) goto decode_next_byte;
+                               /* Subtract the 1 copy we'd output anyway to get extras */
+-
+                               --bd->writeCopies;
+                       }
+               }
+               /* Decompression of this block completed successfully */
+-
+               bd->writeCRC = ~bd->writeCRC;
+               bd->totalCRC = ((bd->totalCRC << 1) | (bd->totalCRC >> 31)) ^ bd->writeCRC;
+               /* If this block had a CRC error, force file level CRC error. */
+-
+               if (bd->writeCRC != bd->headerCRC) {
+                       bd->totalCRC = bd->headerCRC + 1;
+                       return RETVAL_LAST_BLOCK;
+@@ -619,7 +566,6 @@
+       /* Refill the intermediate buffer by Huffman-decoding next block of input */
+       /* (previous is just a convenient unused temp variable here) */
+-
+       previous = get_next_block(bd);
+       if (previous) {
+               bd->writeCount = previous;
+@@ -631,7 +577,6 @@
+       goto decode_next_byte;
+ }
+-
+ /* Allocate the structure, read file header.  If in_fd==-1, inbuf must contain
+    a complete bunzip file (len bytes long).  If in_fd!=-1, inbuf and len are
+    ignored, and data is read from file handle into temporary buffer. */
+@@ -639,7 +584,6 @@
+ /* Because bunzip2 is used for help text unpacking, and because bb_show_usage()
+    should work for NOFORK applets too, we must be extremely careful to not leak
+    any allocations! */
+-
+ int start_bunzip(bunzip_data **bdp, int in_fd, const unsigned char *inbuf,
+                                               int len)
+ {
+@@ -650,16 +594,13 @@
+       };
+       /* Figure out how much data to allocate */
+-
+       i = sizeof(bunzip_data);
+       if (in_fd != -1) i += IOBUF_SIZE;
+       /* Allocate bunzip_data.  Most fields initialize to zero. */
+-
+       bd = *bdp = xzalloc(i);
+       /* Setup input buffer */
+-
+       bd->in_fd = in_fd;
+       if (-1 == in_fd) {
+               /* in this case, bd->inbuf is read-only */
+@@ -669,22 +610,18 @@
+               bd->inbuf = (unsigned char *)(bd + 1);
+       /* Init the CRC32 table (big endian) */
+-
+       crc32_filltable(bd->crc32Table, 1);
+       /* Setup for I/O error handling via longjmp */
+-
+       i = setjmp(bd->jmpbuf);
+       if (i) return i;
+       /* Ensure that file starts with "BZh['1'-'9']." */
+-
+       i = get_bits(bd, 32);
+       if ((unsigned)(i - BZh0 - 1) >= 9) return RETVAL_NOT_BZIP_DATA;
+-      /* Fourth byte (ascii '1'-'9'), indicates block size in units of 100k of
++      /* Fourth byte (ascii '1'-'9') indicates block size in units of 100k of
+          uncompressed data.  Allocate intermediate buffer for block. */
+-
+       bd->dbufSize = 100000 * (i - BZh0);
+       /* Cannot use xmalloc - may leak bd in NOFORK case! */
+@@ -704,7 +641,6 @@
+ /* Decompress src_fd to dst_fd.  Stops at end of bzip data, not end of file. */
+-
+ USE_DESKTOP(long long) int
+ unpack_bz2_stream(int src_fd, int dst_fd)
+ {
+@@ -761,9 +697,9 @@
+       char c;
+       if (i < 0)
+-              fprintf(stderr,"%s\n", bunzip_errors[-i]);
++              fprintf(stderr, "%s\n", bunzip_errors[-i]);
+       else if (read(STDIN_FILENO, &c, 1))
+-              fprintf(stderr,"Trailing garbage ignored\n");
++              fprintf(stderr, "Trailing garbage ignored\n");
+       return -i;
+ }
+ #endif
diff --git a/package/busybox/busybox-1.11.0-ip.patch b/package/busybox/busybox-1.11.0-ip.patch
new file mode 100644 (file)
index 0000000..0cac617
--- /dev/null
@@ -0,0 +1,12 @@
+--- busybox-1.11.0/libbb/print_flags.c Wed Jun 25 14:51:32 2008
++++ busybox-1.11.0-ip/libbb/print_flags.c      Fri Jun 27 00:39:16 2008
+@@ -19,8 +19,8 @@
+                               labels);
+                       need_separator = separator;
+                       flags &= ~ *masks;
+-                      masks++;
+               }
++              masks++;
+               labels += strlen(labels) + 1;
+       }
+       return flags;
diff --git a/package/busybox/busybox-1.11.0-vi.patch b/package/busybox/busybox-1.11.0-vi.patch
new file mode 100644 (file)
index 0000000..cbc5693
--- /dev/null
@@ -0,0 +1,11 @@
+--- busybox-1.11.0/editors/vi.c        Wed Jun 25 14:51:37 2008
++++ busybox-1.11.0-vi/editors/vi.c     Fri Jun 27 06:06:54 2008
+@@ -1894,7 +1894,7 @@
+               p           = new_text + (p           - text);
+               text = new_text;
+       }
+-      memmove(p + size, p, end - p);
++      memmove(p + size, p, end - size - p);
+       memset(p, ' ', size);   // clear new hole
+       file_modified++;
+       return p;