When a size of 0 is passed to copyfd_size, dont do anything, it was
reading untill eof which breaks tar

Fix warning

As we no longer use function pointers for read in common archiving code
archive_xread can be replaced with bb_full_read, and archive_copy_file
with bb_copyfd*
bb_copyfd is split into two functions bb_copyfd_size and bb_copyfd_eof,
they share a common backend.

Use safe read

Fix warning

Make use of libbb functions bb_xopen, bb_full_read, used #define's
instead of static consts, avoid xmalloc.

Remove unused function

Dont over-read fd, use function pointer for hash function.

Fix tar-handles-nested-exclude testcase

Update status of tests which should pass

remove controversial tests

Check at least one context is specified

Check there are files to add the archive before removing a specified
tar file.

Dont attempt to unlink directories

Superficial changes

Fix up the -s option, and make usage consitent with behaviour,
patch by Steven Scholz

Include stdint.h

woops, we needed that function

COMPRESS, not UNCOMPRESS

Update usage for tar -Z

Update md5sum, sha1sum options, patch by Steven Scholz

tar -Z, uncompress support

Dont close original file handle, we may need it later.

Make unlink old files default behaviour and add a new option -k to
prevent overwritting existing files

Only use getopt and associated flags if checking is enabled

Replaced by md5_sha1_sum.c

Replaced by md5_sha1_sum.c

Dont free filename, its needed in the extracted files list.

This was made obsolete by config-udeb-linux-i386

Remove net-udeb flavour, we dotn use it

Use the PROG variable instead of 'busybox' in the release target

Fix a bug where cpio wouldnt work unless -u was specified

Move from read_gz to the pipe()+fork() method.
open_transformer(), common code for pipe+fork.
Function pointer for read() no longer needed.
Allow inflate to be initialised with a specified buffer size to avoid
over-reading.
Reset static variables in inflate_get_next_window to fix a bug where
only the first file in a .zip would be be extracted.

Fix memory leaks

Fix a "broken pipe" problem. vodz, last_patch_116-2

Keep trying to find a good header, if we exit it will cause .tar.gz
files to compute incorrect crc and length for gzip

Catch unsupported features

NEW APPLET: pipe_progress, used by debian installer

Fix build error with tar -j

Remove some tar_gz stuff that get dragged in

Remove debugging noise.

Read in blocks rather than one char at a time, greatly improves speed

Oskar Liljeblad writes:

Here's a fix for the hard-coded device name in fbset.

Michael Smith writes:

I noticed a problem with ifconfig in busybox 0.60.5. The matching code
seems to exist in busybox CVS as well, so I'll paste in the patch that
fixed it.

I was running:
 # ifconfig wan0 1.2.3.4 pointopoint 1.2.3.5 netmask 255.255.255.255

I was seeing the inet addr and P-t-P addr both being set to 1.2.3.5
(the pointopoint address).

wan0      Link encap:Point-Point Protocol
          inet addr:1.2.3.5  P-t-P:1.2.3.5  Mask:255.255.255.255
          UP POINTOPOINT RUNNING NOARP  MTU:1500  Metric:1
          ...

The patch below seems to fix it.

Ulrich Marx writes:

hello

i had some trouble with the filedescriptor in udhcp.
Two things happened on my device:
1.) broken or not connected cable (no dhcp-server)
2.) daemonizing (starting with option -b)

i got a filedescriptor fd=0 from function raw_socket,
after daemonizing (daemon call) the fd is closed.
Client can't recieve data's anymore.

i fixed this problem (like pidfile handling):

Steven Seeger writes:

Hey guys. I've found a bug in modprobe where it generates bad strings and
makes sytem calls with them. The following patch seems to have fixed the
problem. It is rather inherited elsewhere, as there seems to be incorrect
entries in the list which results in more dependencies than really exist for
a given call to mod_process. But, this patch prevents the bad text from
going to the screen. You will notice there are cases where lcmd goes
unmodified before calling system.

Please consider the following patch.

Thanks.

-Steve

Vladimir N. Oleynik (vodz) writes:

Hi Glenn.

I analysed BSS size gzip applet and found may be mistake:
updcrc() checking  if (crc_table_empty) but not resetted this var.
This do make slow gzip applet ;-)

--w
vodz

Marc Kleine-Budde noticed a missing semicolon

Merge common parts of sha1sum and md5sum, which is everything except the
algorithms.
Move algorithms to hash_fd and make them available via a common
function.

Add some basic cvs usage info

Rob Landley writes:

Remove three entries in defconfig that config no longer has.

Rob

Patch from Steven Scholz to send the output from 'time'
to stderr, rather than stdout, so that things like

~ # time bunzip2 -c /tmp/test.bz2 > /dev/null
real    0m 29.44s
user    0m 29.30s
sys     0m 0.12s

operate as expected.

This appears to be the correct fix to make CONFIG_FEATURE_DEB_TAR_BZ2
support compile.  Glenn, you may want to verify this.
 -Erik

fix comment

wrap bb help output if CONFIG_FEATURE_AUTOWIDTH

As vodz pointed out, setting unset_env to NULL is sufficient

Rewrite, 800+ bytes smaller and more robust.

By popular demand, revert to version 1.21, since Rules.mak:1.22
causes the build to ignore CROSS_COMPILER_PREFIX set in .config

Fix tar -j support
Use the old fork() method of tar compression support, rather than
read_bz2....
 - (*uncompress)(int in, int out) seems like a more natural interface
for compression code.
 - it might improve performance by seperating the work into one cpu
bound and one io bound process.
 - There is extra code required to do read_[gz|bunzip] since (*uncompress)(int in,
int out) will normally be used by the standalone compression applet.

There have been problems with this method so if you see a "Short read"
error let me know.

Fix oversight with CONFIG_FEATURE_AUTOWIDTH handling

Set unset_env pointing to a NULL item, to prevent us from walking
off the end of the list and segfaulting.
 -Erik

Fix rdate and ftpget/ftpput so they compile with the new xconnect.
I have checked rdate.  Someone should also check ftpget/ftpput to
be sure they still work.

Avoid conflicts with the 2.6 kernel headers, which define
_IOR rather differently, thereby breaking the BLKGETSIZE64
ioctl.
 -Erik

Rework wget, the xconnect interface, and its various clients
in order to fix the problems with round robin DNS reported
by Andrew Flegg:
    http://busybox.net/lists/busybox/2003-October/009579.html

This removes the ipv6 specific xconnect dns lookups.  I do
not see why that would need to be special cased for ipv6 as
was done, but that will just have to be tested.

So IPV6 people -- please test this change!

 -Erik

Put back the tar support stubs (and warnings) for now.

Try to make indent formatting less horrible

Patch from David Meggy to make the swap default to the new version if no
version is specified and the kernel is relatively new.

Move the tail -c option from FANCY_TAIL to the default tail, this makes
the default tail options conform to SUSv3, with the non-SUS
option all in FANCY_TAIL

Fix stupid typo

Remove config options for sort -u and sort -r, they are always enabled

Add a comment with link to the SUSv3 standard for xargs

run through indent

Patch from Dmitry Zakharov,
Fixes two bugs:
- END block didn't execute after an exit() call
- huge memory consumption and performance degradation on large input
(now performance is comparable to gawk)

make certain values specified in the environment always win

Accept the -c option and do nothing

fix a bug where `which' doesn't check whether the file passed as an argument
is a regular file, patch by Arthur Othieno

make CONFIG_FEATURE_UNARCHIVE_TAPE common between itar and cpio, patch
by Arthur Othieno

"A few references to sourcefiles and URLs in docs/contributing.txt had
gone stale" - Peter Korsgaard

Use the return value from uncompress_bunzip, fix some typo

Add some error messages, use xmalloc instead of malloc

Fix a logic error, the old bunzip code returned non-zero for success,
new code returns 0 for success.

Conditionally compile some files.
This hides a bug related to the new bunzip code in the tar and dpkg[-deb]
applets.
It will also reduce compile time a little as some unused files wont be
compiled.

Another bzip2 update and speedup from Manuel Novoa III, with some
additional changes (primarily lots of comments) from Rob Landley.

the dc applet now needs libm
 -Erik

sigh

Goetz Bock writes:

Dear list,

during my quest do pack busybox into an RPM, I've fixed a small bug
(missing \n) in dc's usage. And added two additional operations: mod and
exp/power.

Feel free to drop them.

last_patch116 from vodz:

Stephane,

>Using busybox+uclibc, crond syslog messages look like:
>
>Oct  9 09:04:46 soekris cron.notice crond[347]: ^Icrond 2.3.2 dillon,
>started, log level 8

Thanks for testing.

>The attached patch corrects the problem.

Your patch is not correct.
Correct patch attached.

Also. Last patch have
- add "Broken pipe" message to ash.c
- busybox ash synced with dash_0.4.18

--w
vodz

Tomasz Motylewski reported that the 'which' applet does not find
files when the full file PATH is specified.

This patch from Arthur Othieno fixes it.

Patch from Arthur Othieno for style-guide.txt conformance

Patch from Arthur Othieno for docs/style-guide.txt conformance.

Patch from Arthur Othieno to fix a compiler warning

make this a little bit less messy

Update mail location

Disable any buffering to stdout

Tito writes:

Hi to all,
I'm sorry but  I didn't spot this big fat bug until now,
Matteo Croce emailed me about it.
Please apply this patch as the devfsd applet is broken
and works only on a system booted with a standard devfsd
( the test I mostly did :-(   ), but if used at boot time
it DOESN'T WORK.

Thanks in advance and please apply
Tito

Andreas Mohr writes:

the busybox menuconfig triggered my "inacceptable number of spelling mistakes"
upper level, so I decided to make a patch ;-)

I also improved some wording to describe some things in a better way.

Many thanks for an incredible piece of software!

Andreas Mohr, random OSS developer

Do a better job of dealing with screwy s390 abi changes

ccording to this:
    http://sources.redhat.com/ml/binutils/2003-01/msg00290.html
The name R_390_GOTOFF was changed to R_390_GOTOFF32.
 -Erik

Manuel Novoa III writes:

Hello Rob,

Here's a patch to your bunzip-3.c file.  Nice work btw.

One minor bug fix... checking for error return when read()ing.
Some size/performance optimizations as well.  One instance of
memset() seems unnecssary.  You might want to take a look.

Anyway, on my machine, decompressing linux-2.6.0-test7.tar.bz2
to /dev/null gave the following times:

        bunzip-3.c    bzcat (system)   bunzip-3.c (patched)
real    0m24.420s     0m22.725s        0m20.701s
user    0m23.930s     0m22.170s        0m20.180s
sys     0m0.070s      0m0.080s         0m0.140s

Size of the patched version is comparable (slightly larger or
smaller depending on compiler flags).

Manuel

Rob Landley's new micro-bunzip version 3.  Rob writes:

The API for using partial writes, as described in my last message, sucked.

So here's a patch against my last patch that changes things so that
write_bunzip_data calls read_bunzip_data itself behind the scenes whenever
necessary.  So usage is now just start_bunzip(), write_bunzip_data() until it
returns a negative number, and then the cleanup at the end of
uncompressStream.

It adds 32 bytes to the executable, but it should allow the caller (tar) to be
simplified enough to compensate.  Total -Os stripped exe size now 6856 bytes.

Rob

P.S.  I attached the whole C file so you don't have to keep incremental
patches straight if you don't want to. :)

P.S.  In the version I'm banging on now, I've simplified the license to just
LGPL.  I read the OSL a bit more closely and the patent termination clause
would have bit IBM in their counter-suit of SCO if the code in question had
been OSL instead of GPL, and I've decided I just don't want to beta-test
legal code right now.