Add missing include files

passwd and sulogin also need libcrypt, via libbb/pw_encrypt()

s/fileno\(stdin\)/STDIN_FILENO/g
s/fileno\(stdout\)/STDOUT_FILENO/g

Vladimir N. Oleynik (vodz) writes:

Ok. Last patch reduce 73 bytes for compensate (and over) your changes ;-)

Comments:
Added cin_fileno variable, auto setted to 0 from BSS and have "eq" stdin
descriptor if isatty(stout)==0, removed global variable FILE* cin.
Removed default setting to terminal_width/terminal_height, this used
only from main() and setted after call get_terminal_width_height()
always correct.
Variable please_display_more_prompt changed to bits logic, have size
reducing.
--w
vodz

Update docs

Brian Pomerantz writes:

I've noticed a bug in the "autowidth" feature more, and is probably in
others.  The call to the function get_terminal_width_height() passes
in a file descriptor but that file descriptor is never used, instead
the ioctl() is called with 0.  In more_main() the call to
get_terminal_width_height() passes 0 as the file descriptor instead of
fileno(cin).  This isn't a problem when you more a file (e.g. "more
/etc/passwd") but when you pipe a file to it (e.g. "cat /etc/passwd |
more") the size of the terminal cannot be determined because file
descriptor 0 is not a terminal.  The fix is simple, I've attached a
patch for more.c and get_terminal_width_height.c.

BAPper

Some corrections from vodz:
- Make -u/-l mutually exclusive
- Minor size reduction

The utc variable was not modified according to the -u/-l command line
parameters.

Tito noticed a printf that should have been a bb_error_msg.

Oops.. got a bit to aggressive with size optimization and global replace.  :-(

As noted in a patch from Kendrick Hamilton, rmmod was only
half way converted, and still used the old delete_module(),
call rather than a syscall, in one spot.

Add missing ELFCLASSM for m68k

Only use R_68K_GOTOFF if it is defined

Fix broken arg parsing (was not passing pointer to items so p, argc, and argv
were only modified locally).  Fix error reporting to properly describe why
ioctls fail.

Patch from vodz to fix the dynamic vars patch, which I should not
have checked in.  Vladimir writes:

Your patch have many problem.
1. You always added + time(). This cannot reset RANDOM=value for debuging
with
replay sequential.
2. Hmm. I examine bash 2.04 source. This pseudorandom generator use low bits
of
counter value. You use high bits. This make bad pseudorandom values after
have
0-value. For example, if + time() do remove, your generator always return 0
after
first generate 0.
3. Memory leak per call. Use ash-unlike unecessary bb_strdup function.
4. Unsupport show last $RANDOM value for "set" and "export" command.
5. Bloat code. Busybox-unlike patch - added unstandart feature as default
hardcode.

Last patch attached.

Erik, why you apply Paul patch with have 5-th point problem? :(

Last patch have ash change xwrite() to fresh libbb/bb_full_write interfase
(haved loop after EINTR).

--w
vodz

Oops.  Using the wrong variable was a rather stupid
thing for me to do.

Remove trailing whitespace.  Update copyright to include 2004.

I redid route.

Update docs for start_stop_daemon to match reality.  Update
the reality a bit to better match debian behavior.

Fix some doc generation problems

Hideki IWAMOTO writes:

Current `tr' implementation has a problem, if `plain char' is signed.

[current cvs version]

>echo a | _install/usr/bin/tr '\0' '\377'
Segmentation fault (core dumped)

[patched version]

>echo a | _install/usr/bin/tr '\0' '\377'
a

Fix some goofy formatting

Add prototypes for safe_strtol and friends

Fix awk entry, which was not showing up in BusyBox.txt for some reason

Paul Mundt, lethal at linux-sh dot org writes:

Here's a follow-up replacement to the patch I sent earlier, this adjusts some
of the semantics of the dynamic variable setting. Namely, dynamic vars can hook
a set handler (which RANDOM uses to adjust the seed). They'll only lose their
dynamic status if they're unset.

I've used the same approach that bash does to come up with the random number,
mostly just for consistency.

For example:

$ echo $RANDOM
13759
$ echo $RANDOM
20057
$ echo $RANDOM
1502
$ export RANDOM=42
$ echo $RANDOM
24179
$ echo $RANDOM
2046
$ unset RANDOM
$ echo $RANDOM

$ export RANDOM=42
$ echo $RANDOM
42
$

Patch from Thomas Frohlich to fix an option ordering bug of mine.

When displaying the size in 1kB blocks round up if an odd number of
blocks

Patch by Seth W. Klein, the -l switch was reversed

In spite of the feature freeze, check in a complete rework of route which
fixes some bugs, adds some error checking, and removes _lots_ of bloat.
Text size on i386...
              old     new
   ipv6      5425    3523
   no ipv6   3143    2193

Note that /proc must be mounted for filesystem type autodetection
to work.  Without /proc mounted, one must explicitly specify the
type of every filesystem being mounted.

David Anders (prpplague) submitted this patch to allow login to work
when the device nodes are symlinks on a read only file system.

Fix broken sort order flags.

My bug.  :-(

Fix/eliminate use of atol

Patch from Matt Kraai to fix debian bug number 231994.
There was an extra blank line preceding the first directory.

Woops, how did that get in there

Don't build these support functions for uClibc, as it always supplies
versions.

Patch from John Powers, adds multicast (rfc2090) and timeout (rfc2349)
options

Patch from duane no-name, optionally request additional items

Patch by Andrew Victor,
In arpping.h, fix structure alignment of "struct arpMsg".
GCC can insert padding in the structure which causes udhcpd to send an
invalid ARP packet on the network.  It will then not receive a valid
reply, which can cause it to assign an IP address that's already in use
on the network.
(With kernels before 2.4.20, the "struct ethhdr" in linux/if_ether.h
wasn't marked as packed.  This is also an issue if your toolchain was
built with a pre-2.4.20 kernel).

Fix a typo, patch by Andrew Victor

Patch from Tony J. White to allow paths to be configured

Patch by Denis Vlasenko to add forground support.

Fix debugging output, patch by Hideki IWAMOTO

Patch by Felipe Kellermann to fix a compiler compatability problem

Patch from Steven Scholz to fix compiler warnings

New applet, eject, by Peter Willis

Close bracket in description from Peter Willis's eject applet

Patch from Vernon Sauder.
The off_t type is not a consistent size; it depends on the kernel options
(something about large file support). Therefore, the format string for
printing an address is not always the same.

Fix a compile error identiefied by Steven Scholz.
Declare variables first for compatability.

Check file has execute permission for the current user, minor formating

Quick fix for tftp truncation bug

Patch from Denis Dowling to fix a ocuple of usage entries

Fix option handling, -i or -t must be given, if both ignore the -t.
Use bb_getopt_ulflags.

Joe.C, joe at numa dot com dot tw writes:

Hi,

When downloading files over slow network (e.g. wireless/
internet) using IE, sometimes it will stop downloading and
show error message 'connection closed' when the download
is almost complete. This is because IE can't handle server
close connection properly.

Apache http_main.c fix this problem by close the connection
after client close the connection. This patch do exactly the
same thing. Please consider include this patch.

Joe.C

Prepare for release

bb_full_fd_action was incorrectly returning an error message causing
major breaking.

Bump up version in preparation for -pre8

Fernando Silveira writes:

Hi,

  Well, I made this patch a long time ago (08/2002) because it was a
  need of a project, but had no time to send it to you. It adds support
  to `autologin' option of the telnet protocol. It has been used since
  made with busybox 0.60.3 at production and I had no problems with it.
  I have ported it to the HEAD revision of the CVS server (20040211) and
  I hope you enjoy and apply it to the official sources. :)

Thanks a lot!

Fix from Paul Kortekaas for syslog -C option, size should be in kB.

Patch from Dmitry Zakharov, this line was missedfrom the last patch

For the time being, revert the changes for detecting copying
a directory into itself.  It is harder to do this correctly
than it appears.  Not trying at all seems a better compromise
for the time being, untill we can implement this correctly.

Patch from Chris Larson (kergoth), to allow multiple directores to be
unmounted at once.

Patch from Vodz, cleanup memory usage, send strdup error messages to
syslog.

Vodz. last_patch_128
- declare applet_using as static from applets.c
- small correction to cmdedit,
   previous version cleared history after Ctrl-C
- small spelling correction (by Friedrich Lobenstock)

Patch from James Zhu, telnetd window resizing support.

Use bb_getopt_ulflags, marginal saving, better argument checking.

Use /var/lib/hwclock for adjtime, its really a state file rather than a
config file, so it should be in /etc, FHS mentions it also.

Patch from Florian Schirmer. compile fix for debug builds

Patch from Hideki IWAMOTO, output debugging messages to stderr

remove duplicate header declarations

Fixup braces (patch from Thomas Geulig), run through indent and manually
adjust.

Thomas Geulig, remove duplicate header

Update usage for ftpget and ftpput

Ptch by Hideki IWAMOTO, fix a bug preventing tftp from getting or
putting more than 0xffff blocks.

Return 1 upon failure

No need to check proxy env variable twice, ignore proxy env variable if
its set to 0

Sometimes i get carried away with the use of function pointers, im sure
it seemed like a good idea at the time.

Use return instead of exit, use == instead of & ==, left justify labels,
adjustment of whitespace.

Document missing cp options

Fix up hard links

A strict interpretation of the ustar format requires the type flag to be
interpreted, we cannot depend on the file type being set in the mode
field.

Add extra comments, freakout if a pax archive is encoutered.

Chris Larson (kergoth) writes:

As Manuel points out, this is a flawed fix, and doesnt fix the
following:

mkdir -p cpa cpb cpc
cp -a cpa cpa/cpb/cpc

Attached what appears to be a more sane fix.  Apply on top of previous.
Please confirm sanity.

Chris Larson (kergoth) writes:

I was adding -s/--symbolic-link support to busybox cp when I noticed a
bug with -r/-a.  Test case:

mkdir -p test/out
cd test
busybox cp -a * out/

Will never return until we run out of open files or similar.
Coreutils cp on the other hand will error with "cannot copy a directory,
`out', into itself, `out'".  Patch attached.

Dont use same variable and struct name

Formatting

remove debugging

Patch from OpenWrt.

Add -i option to sed, to edit files in-place.

Fix vstype[] to match VS* defines.  Patch from OpenWrt by mbm.

Remove the unused CONFIG_FEATURE_HTTPD_SET_CGI_VARS_TO_ENV option.

Mark Lord writes:

The config file processing for inetd in busybox-1.00-pre7 and newer
is broken -- the saved command argv[] values overwrite each other.

Patch to fix it is attached.

Cheers
--
Mark Lord
Real-Time Remedies Inc.

Add the -r option, patch from Rob with some help from myself.

Remove debugging

Dont strip trailing '/' until _after_ i test to set if its there !

Patch from Florian Schirmer <jolt@tuxbox.org>.  When I rewrote the parser,
I overlooked the case of someone wanting to use a valid but empty suid
config file.

Woops, im getting ahead of myself, we dont have the -r (refernece)
option yet

define option names to be clearer, simplify nested if statements, remove
un-needed if statement, minor indenting change

Avoid naming conflict with symbol in newer glibc headers