Don't return 0 when there's an error.

With -a, the jump to mount_it_now can skip the initialization of f, and
if we don't zero it after closing it we re-close a filehandle that isn't
open, and since this is a file _pointer_ it segfaults on a double free.

Yeah, subtle bug.  I need to break this out into separate functions if I can
figure out how to avoid making the code larger while doing so.  Part of
the general -a and -o remount work I need to do, but that's after 1.1.0...

Both atime and diratime should switch _off_ the corresponding no* flag.

upgrade ZyXEL from hall of shame to hall of [product] fame

removed a warning about an unused pargv in bb_getopt_ulflags. Noticed by Tito

Bug 112, return value of awk should be /256.  In reality, we should probably
be using WEXITSTATUS(), but until I can figure out why the heck that would
want to do (*(int *) &(status)) on the value, I'm happy just fixing the bug we
actually see.

Fix bug 603: "chown :root thingy" should work now.

Bug 547: writing out the tar file header before we confirm we can actually
open and read from the file isn't something we can recover from after the
fact.  Resequence things to check first, write second.

Minor cosmetic fix from Tito.

Why we didn't take a watchdog.c suggestion this time 'round, in case anybody
comes up with a fix...

declare const for fun

Tito writes:
Somehow while applying the bb_do_delay patch a change slipped
in libbb.h that broke compilation.

libbb.h Line 355
  extern char bb_path_mtab_file[];

This conflicts with mtab_file.c
#if defined(CONFIG_FEATURE_MTAB_SUPPORT)
const char bb_path_mtab_file[] = "/etc/mtab";
#else
const char bb_path_mtab_file[] = "/proc/mounts";
#endif

patch from tito: consolidate delay functions as bb_do_delay()

Bug 624 wants quoted char support for printf, so you can do something like:
  printf '%d\n' '"x"'
and have it print out 120.  This is the smallest implementation I can think
of at the moment.

No, we _want_ the suid and sgid bits reverted by chown, for security reasons.
They mean something different when the user they're switching to is different,
so if you still want suid you reset it afterwards.  This is a safety feature.

- better wording

use MAP_PRIVATE instead of MAP_SHARED for uClinux systems (see Bug 550)

Jason Schoon writes:
Here is a patch for kill that I posted long ago, but forgot about
until today.  This allows kill to specify a negative process/group
(such as -1).  The shell already had this fix applied by Vodz some
time ago.

import nios2 support from microtronix

make mount usage a bit more terse

just use bb_xfopen()

fix comment documentation

- silence warning about unused param
  needs revisit later on like many others to use ATTRIBUTE_UNUSED from platform.h

- use HOSTCC instead of a hardcoded cc

remove buffer overflow by Erik and decodeString problem by Glenn, add error check of decodeString as Apache

When you went "losetup -d /dev/loop0 /dev/loop1" the error message was strange.

Trying to losetup a device as a regular user shouldn't result in an endless
loop, and the error messages should display correctly now.

Second attempt to simplify the login menu.  Shadow password stuff at the top,
with no dependencies (since depending on 9 of the 13 apps in the menu gets
a bit silly).  And fix my attempt to make internal shadow password support
automatically depend on internal password support plus shadow password
support.

- fix check for group and user, remove some whitespace while at it.

Fix chroot, leave console alone if -c not specified, and yank debug code.
(I still haven't set up a test environment to confirm this works...)

Some minor cleanups/bugfixes split off from the big remount work:
  Shorten GPL boilerplate.
  Enabling FEATURE_CLEAN_UP broke things in two places.
  Move the NFS with uClibc check to nfsmount.c

Shadow password support went beyond the dependency event horizon, just make
it an independent menu.  And make internal shadow password support a subset
of the other internal password function support.

Actually, static buffers look like _this_...

remove debug feature for production

Make a few warnings go away.

Do an svn log -v, the information in this changelog is stunningly
obsolete.

Shadow password support involves vlock too.

Patch from Tito to make vlock much smaller.

strncmp isn't automatically superior to strcmp.  Especially when it isn't
used right.

Sigh.  Left the debug path in.  Fixed.

Added /etc/mdev.conf support.  Adds about 1.9k to mdev.

Bug 601: When we fork an archiver and feed stuff to it through a pipe, if
we don't close the pipe the child process won't exit, and we'll hang in
waitpid().

Ahem.  Vladimir checked in a different fix bug didn't close out the bug.
Reverting...

Less would segfault if it had no tty.  (Bug 600.)

I screwed up the last commit: if dt is null when ENABLE_MULTIPLE_OPTIONS is
off, we'd dereference the null.  Oops.

Stephane Billiart said:

the attached patch makes applet list in busybox.links
correctly include tr and watchdog.
Currently, they don't appear because they are prefixed
with ENABLE_ in include/applets.h

sync license from mainstream

fopen(TTY...) to bb_xfopen(), close bug 600

- Stephane Billiart writes:
  Fix the compilation of head and tail.

  thanks!

- remove duplicate entry for switch_root

Add a fflush(stdout).

Better use of the ENABLE guards.

The rest of Yann E. Morin's install revamp.

Eliminate unnecessary allocation from md5sum that was leaked without ever being used.

Fix -g and -p options.  Patch from Traceman.  (I don't know who that is,
it's the darn bug generator again.)

Patch from dlively (whoever that is, see bug 591) to fix
"ip link set <dev> addr".

- several functions depend on IPv4 or IPv6 od IPX support.
  removes warning about unused code for allbareconfig.

- remove warning (thanks Yann E. MORIN) and switch to ENABLE_
- typo: s/begining/beginning/g

remove sigsetmask, noticed by Bernhard

- check if CC supports -funsigned-char
- add CC_MAJOR and CC_MINOR variables. (gcc centric for now)

- wrap label GET_COUNT in #if ENABLE_FEATURE_FANCY_applet to avoid warning
  about unused label.

- do away with superfluous keyword.

remove strange bloat dup2() usage, sync this code with mainstream login-utils, size reduce

Nothing to see here.  Move along.

Not buying it, eh?

I know I said new features before 1.1, but, well...  (I was weak!)

The config file and hotplug modes aren't implemented yet.  Might take a stab at
those tomorrow.  (I _should_ go back to focusing on the bug triage list.)

Minor fix: if(CONFIG) breaks the build when that CONFIG is disabled, it has
to be if(ENABLE).  (Make allbareconfig is a good testing thing.)

Patch from Yann E. Morin, something to do with bugs 276 and 272.

- typo: s/sucess/success/g
  What's up with loginutils/su.c line 42: "SYSLOG_SUCESS" ? Please have a look..

- fix zcip for IMA compile mode.
  rename usage() to zcip_usage, make struct arp_packet static. This applet is
  not ISO C89 compatible due to the use of C99-style comments ("c++ comments).

Flush output to log promptly (bug 389).

Flush input when prompting for a password (bug 373).

Bug 310: be consistent about stat/lstat and chown/lchown to avoid accidentally
modifying permissions or ownership when twiddling symlinks.

Uninitialized variable causing intermittent syntax errors (bug 449).

printf() arguments shouldn't be passed straight from user supplied data.
(Security thingy.)

Change CONFIG_MODPROBE_MULTIPOLE_OPTIONS to
CONFIG_FEATURE_MODPROBE_MULTIPLE_OPTIONS.

Patch from Yann E. Morin to reorganize module config options.

Make insmod quiet by default (patch from Yann E. Morin).

Yann E. Morin's update to modprobe usage.

Patch from Yann E. Morin to make rmmod report failure properly.

Add build options to control SuS compatability, allows numeric
option handling to be disabled.
Defaults to enabled, so no changes in default behaviour

- remove unused variable have_read_stdin
- use FLAG_WIDTH

avoid mixed declarations, to support older compilers just a little longer.

Macro was wrong, name(void) vs name(x), and apparently gcc 2.95 cares about
matching up macro arguments even when the macro substitutes to nothing...

uClibc can't handle anonymous MAP_SHARED.  Since we're treating it as
read only data anyway, MAP_PRIVATE shouldn't make a major difference.

Tito is using broken kernel headers that pollute the namespace with every
CONFIG_ symbol in the kernel, and this clashes with busybox's CONFIG_TR
and CONFIG_WATCHDOG, causing applets.h to barf if they're not switched
on (since the broken headers don't affect kconfig or the makefiles).

Since such broken kernel headers are common enough to crop up every few
months, a simple work around is to move TR and WATCHDOG from CONFIG_
to ENABLE_ early.

reduce 3 warning if compile with -W

restore compare_string_array new interface (make broken by landley)

Install links patch from Yann E. Morin.  (Another thing hanging around in my
tree forever.  Tweaked the docs a bit.)

If we really _need_ an #ifdef like this, then we should be using the libbb
getopt infrastructure instead.

Another cleanup patch that's been in my tree for a while.  Again I think it's
originally from Bernhard Fischer...

A cleanup patch I've had lying around in my tree for a while, I think it
was from Bernhard Fischer?

reduce signedness warning

reduce signedness warning

remove warnings if compile with -W, use ENABLE_FEATURE vs CONFIG_FEATURE

Make char default to "unsigned" on all platforms so we have a consistent base
to deal with all the new gcc 4.0.2 warnings from.

Fix losetup so that it A) actually works again, B) has much better error
messages, C) can show the current association (if any) when called
with only one argument.  Update the documentation a lot too.

Remind me to add a test suite for this thing.  I think I've figured out
how to handle root-only testsuites...

getopt -> getopt_ulflags. noticed by Rob Sullivan

update license

ShaneAnderson writes in Bug 565:
need to include .config file before setting CROSS otherwise
TARGET_ARCH isnt set properly

add "-C" format to hexdump

fixes bug #113 and satisfies a personal need at the same time.

output compares identically to util-linux version.  (with
exception of whitespace differences on last lines of output with
non-uniform length, which are neither fixed nor worsened by this
change.)

fix bug #474:
     0000474: vi crashes often

problem was that the buffer used for "." command ("last_modifying_cmd")
wasn't being maintined correctly -- the recording code was walking back
over the front of that buffer when a repeatable insert command
included backspacing (e.g. "i\b\b\bfoo").  the fix is to simply
record the backspaces along with the rest of the command.

also, cleaned up start_new_cmd_q() slightly.