mount: style fixlet

style fixes

mount: revert mount --bind to using "bind" as fstype.

mount: mount_it_now() - char *dir is not really needed.

mount: fix "duplicate mount options in mtab" bug

mount: nfs_strerror's static buffer was bigger than needed.

mount: getopt_ulflag'ification

mount: style fixes

mount: fix bugs: free(mp->mnt_fsname) of non-malloced ptr;
check for "more than 2 arguments" was actually checking for -2.

mount: use bb_simplify_path as appropriate

mount: style fixes

login: apply fixes + getopt_ulflag'ification by Bernhard

- Steven Scholz pointed out that ssd's make-pid doesn't take an argument.

mount: reorder things, fix NFS-less mount.

Unbreak allbareconfig.

So, in the cornucopia of superfulous warning directives I didn't add to the
tree, we have the warning about failed inlines (which with our inline
limit set to zero means any usage of the "inline" keyword at all).  Note
that setting the inline limit to zero, and using -Werror, both predated
adding the warning about inlines to the tree.  So whatever checkin added
that did nothing but break the tree.  But oh well.

The second category of superfluous warnings is warning about functions with
no previous declaration.  Apparently, if you add ALWAYS_INLINE to an empty
function definition, it considers the sucker undeclared as far as the
warning is concerned.  (I.E. it's a buggy warning.  I try not to ask the
compiler to generate warnings it can't competently generate.)

This is why I removed "inline" (unbreak allbareconfig), and couldn't replace it
with "ALWAYS_INLINE" (still broke allbareconfig).

Unbreak allbareconfig.

We've had -Werror in the tree for quite a while, so adding #warning
somewhat counterproductve way to make comments on the code that belong on the
mailing list anyway.

Strangely, using // in the comments I added was not a persistent accident.

login: eliminate forward decls and #ifdefs

umount: do not try to close FILE* which is NULL.

mount: -o remount should not add lines to /etc/mtab

mount: move code from nfsmount.c into mount.c

- make sure that the 'build' dir exists even if we are about to build just one applet.

- do away with silly comment and do the Right Thing wrt building the archives.
  You have to provide the absolute path to the objdir/target.ext you want to build, as can be seen in the respective makefiles..

mount: fix mtab support (but it is still rather buggy)

svn 15355 replaced "BusyBox is licensed under the GNU General Public Public
License" with "BusyBox is licensed under the GNU General Public
License version 2 or later... (This is the same license the Linux kernel
is under...)"  Except that the Linux kernel isn't under GPLv2 or later,
it's just under GPLv2.

Now they match again.

More code from kernel developers, and therefore licensed under GPLv2 only.
Clarify the license boilerplate.

On an unrelated note, this could use busyboxification.  check_mount() looks
reusable and ask() is generic...

Add Larry Doolittle's ntpclient, plus some formatting tweaks.

- fix copy'n paste errors that got introduced when switching to the shorter boilerplate.
  No object code changes.

- r16075 broke for de-selected FEATURE_UTMP; Partial fix that wants some more cleanup (see FIXME in the patch).

Revert duplicate patch.

I don't know why patch decided to apply this even though it was already in
the tree.  I thought the other hunks failed because they were totally
unrelated leakage from Bernhard's tree (which they are; was the a reason
for bundling them in with this fix?  Do they have something to do with
the GCC 2.95 fix?  I suspect they prevent me from backporting this
patch to 1.2.2 because the header consolidation into libbb.h hadn't
been done yet, and no I'm not fixing it up: if that's the case then
this patch won't be in 1.2.2 due to extraneous changes bundled with it that
prevent a clean backport without rolling a new patch).

Remove pointless "const".  Bloatcheck says 0 bytes difference.

Fix from Bernhard for somebody trying to limp along with gcc 2.95.4.

Random cleanup hanging around my tree.

- small trivia to be gentle to gcc-2.95.x which had no va_copy but only __va_copy.

- fix warning about discarding qualifiers in initialization

nfsmount: sanitize it. It had a rather peculiar idea of implementing "bg"
option - it was going to return a special flag back to caller and
expecting caller to call it again with special parameter! Also
caller was charged with calling mount() syscall...

mount: mtab support was non-functional. Enabling it revealed serious bug
which is not fixed yet.

- convert a few xopen3(,,0) into xopen(,). Also peruse the fact that xopen defaults to 0777 on it's own, so we don't need to xopen3(,,0777).
  Saves a few bytes:
$ size busybox.old busybox                                text    data     bss     dec     hex filename
 839676    8780  243592 1092048  10a9d0 busybox.old
 839660    8780  243592 1092032  10a9c0 busybox
$ make CC=gcc-4.2-HEAD bloatcheck
function                                             old     new   delta
static.vconfig_main                                  281     279      -2
static.rx_main                                      1076    1074      -2
static.mt_main                                       294     292      -2
static.gzip_main                                     753     751      -2
static.mkfs_minix_main                              4034    4030      -4
static.unzip_main                                   1771    1766      -5
static.crontab_main                                 1513    1507      -6
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/7 up/down: 0/-23)             Total: -23 bytes

- merge xstat.c into xfuncs.c

- Tito pointed out that Rob forgot to add e2fsprogs/blkid/list.c

Build fixes for gcc 4.0 with -Werror, from Tito.

Error reporting fix for sulogin, plus remove help entries for options we never
implemented.  (Plus a bit more of bbsh leaking in from my tree, but it
shouldn't hurt anything and I'm lazy...)

nfsmount: remove some really old code (for kernels 1.x!) + small cleanups.

nfsmount: delete nfsmount.h

nfsmount: nfsmount.h merged into nfsmount.c

nfsmount: style fix

ipcs: silly trick with strings. saves ~200 bytes.

Cleanup headers and replace the zero padding with an even smaller version.
One of the pads turned out to be unnecessary: sizeof(struct TarHeader) is
TAR_BLOCK_SIZE, the padding's in the struct.  The others could be done inline
with bb_common_bufsiz1.

This is a cleanup I did to Denis' patch long ago, but got sidetracked by
what turned into svn 15660.

sulogin: minor cleanup.

setkeycodes: something horrible happened here
to indentation... fixed now

top: fixed wrong comment, made error message more clear

tar: fix "xopen with O_CREAT" warning, improve zero padding write
(was doing zillions of 1-byte write syscalls)

xopen3(O_RDONLY) -> xopen(O_RDONLY).

using [xa]sprintf for string concatenation is neat and saves
~100 bytes according to bloatcheck. Also this fixes bug in rpm

wget: wrap one atrociously long line (around 200 chars long!).

udhcp: FEATURE_UDHCP_SYSLOG should be selected only if
udhcp[cd] really are going to use it.

login: use some ideas from util-linux's login.
O_NONBLOCKing output on login timeout.
fchmod instead of chmod (latter is racy).
is_my_tty() is not needed anymore after race is fixed
(is_my_tty() was racy too anyway...).

few random readability enhansements. No code changes

login: make it saner and smaller by ~0.5k.

login: small fixes like \n removal, bb_error_msg'ification, etc.

login: previous commit comment was wrong :)
That commit added login script support.
Now _this commit_ is a style fix. Sorry....

login: style fixes

Shrinkage/cleanup from Tito.

Fix warnings.

Second drop.  More infrastructure in place, especially for parsing pipelines.
The minimal version got a couple hundred bytes bigger, partly because it's
broken into more functions but mostly because it now dynamically reallocates
and frees everything with no hard-coded limits.  (I thought about making
that optional, but there's a limit to what I can debug and maintain.  It
should still be reasonably NOMMU friendly, the allocations are small and
short lived, and shouldn't contribute noticeably to long-term memory
fragmentation.)

A quick cleanup pass of some low-hanging mess.

Bugfix for: echo '123456789' | sed 's/./|&/5'

Fix (hopefully) bug 976. Need more thorough audit.
Restore erroneously removed FEATURE_UDHCP_SYSLOG.

getty, sulogin: convert to using bb_msg for syslog output

fix if (ENABLE_FEATURE_SYSLOG & (...)) - should be &&

Add CONFIG_FEATURE_SYSLOG which controls whether
bb_xx_msg will ever try to send output to syslog.
Add "select CONFIG_FEATURE_SYSLOG" to relevant applets.
This allows to omit syslog code if we do not have
any syslog-capable applets in the build.

CONFIG_FEATURE_COMPRESS_USAGE was impossible to turn on
if !CONFIG_NITPICK

adding files missed in prev updates (forgot about svn add)

Denis Vlasenko has started to put stuff on his BusyBox home page, so index it.

removed a lot of trailing \n in bb_msg() calls. It is added
automatically by function itself.

Implement optional syslog logging using ordinary
bb_xx_msg calls, and convert networking/* to it.
The rest of bbox will be converted gradually.

- strip 399424 off the bss by making decompress_uncompress buffers config buffers.
  Compile tested (too lazy to look for a small .Z on the net).
$ size busybox.old busybox
   text    data     bss     dec     hex filename
 859555   10232  645732 1515519  171fff busybox.old
 859683   10232  246308 1116223  11083f busybox
$ make bloatcheck
function                                             old     new   delta
uncompress                                          1036    1160    +124
inbuf                                               2116       4   -2112
outbuf                                              4100       4   -4096
htab                                              131072       - -131072
codetab                                           262144       - -262144
------------------------------------------------------------------------------
(add/remove: 0/2 grow/shrink: 1/2 up/down: 124/-399424)   Total: -399300 bytes

- mdev and grep use xregcomp. Closes bug #1021

Vladimir Dronnikov (ybrnj80 at yandex dot ru) pointed out that my cleanup of
his code introduced a bug (an extra backslash in the CIFS mount string).

The bb_ prefixes were removed from xasprintf() and xopen() a month ago (in
svn 15767).

run_shell.c: style fix

xfunc: fix: && -> &. Also nuked two double semicolons...

Might as well commit this to have the history.  It's not linked in to the
applet list yet (and won't be until it can replace lash, I'm not having five
shells in menuconfig at once), but you can build it with scripts/individual
and mostly this is checked in so I can bloatcheck future versions against it
easily....

This is about as small as a shell can get and still be a shell.

Thinko.

Patch from Natanael Copa to make start-stop-daemon just use readlink and
strcmp to check if a program is already running.

sulogin: fix bug: -tNNN didn't work

dpkg: fix buffer overflow (bug 983)

uuencode: common implementation for wget and uuencode (closing bug 694)

archival: added O_TRUNC so that when we overwrite files on unpack,
          we truncate them. Also spotted & fixed hard to trigger bug
          with extension handling.

tar: revert older fix (non-portable), added new one.
     testsuite tar-extracts-all-subdirs now passes.

zcip: bb_error_msg had stray newlines; small optimization in opt parsing

zcip: minor fixes

zcip: fix stdout/err versus syslog output.
      Incomplete: xfunc() would not respect this. TODO.

zcip: getopt -> bb_getopt_ulflags

zcip: convert lots of zero-initialized locals
      in zcip_main() to statics in bss.

zcip: use xfunc() where appropriate

replacing func() with xfunc() where appropriate

zcip: apply patch from
      http://bugs.busybox.net/view.php?id=1005
      zcip does not claim another IP after defending

tar: tar xf foo.tar dir/dir did not extract all subdirs.
Added testsuite entry for this

ping: bind -> xbind