whitespace updates

recode functions to shrink size

setup the HAVE_* defines

About time to just apply this and kill off the patches

Tito posted a devfsd error message fix.  It's highly deprecated and will
presumably be removed eventually (use udev), but as long as it's in there.

Tito says:

The sense of this patch is to call:
read_config_file_err:
#ifdef CONFIG_DEVFSD_VERBOSE
    msg_logger(((optional ==  0 ) && (errno == ENOENT))? DIE : NO_DIE, LOG_ERR, "read config file: %s: %m\n", path);
#else
    if(optional ==  0  && errno == ENOENT)
        exit(EXIT_FAILURE);
#endif

just after the failure of the  call that set errno ( stat and fopen)
to avoid false error messages.

Clean up strings.c to use busybox's option processing.  Bug 006, apparently.

Patch from Dmitry Zakharov:

Charlie Brady wrote:
> Here's another awk parsing problem - unary post increment - pre is fine:
>
>bash-2.05a$ echo 2,3 | gawk -F , '{ $2++ }'
>bash-2.05a$ echo 2,3 | /tmp/busybox/busybox awk -F , '{ $2++ }'
>awk: cmd. line:1: Unexpected token
>
Here's a fix for this. There is another problem with constructions like
"print (A+B) ++C", I don't
know whether somebody uses such constructions (fixing both these
problems would require very
serious change in awk code).

Thus spake Brenda J. Butler:

We were seeing some timeouts when getting files with the busybox tftp
client.

With tcpdump, we saw that the tftp client was receiving blocks and
ack'ing them, but the server was failing to receive the occasional
ack.

When that happened, the server would send the last block over again,
but the tftp client was expecting the next block.

This patch allows the client to recover from this situation
(it sends an ack for the repeat block but does not write it
to the local file).

I hope it meets your approval, please don't hesitate to send
me comments for improvement.

The patch is against "head" in svn, I tested it on an older version
of busybox in our environment.  It applied cleanly to the older
version.

Credit for this goes to my co-worker John McCarthy for finding
it and me for fixing it (assuming it works for everyone else too).

cheerio,
bjb

Shaun Jackman submitted a patch converting an allocation to use
CONFIG_RESERVE_BUFFER.  (Rob Landley removed an #ifdef, per discussion on
the list.)

Patch from Shaun Jackman to save a few bytes.

Tobias Krawutschke found a bug where the DHCP client would accept packets
with the wrong ARP address, meaning we could easily get somebody else's IP.
That is a bad thing, and this is the minimal two-line fix.

Add readprofile applet support.

Patch from Colin Watson (mangled slightly by Rob Landley):

This patch implements the 'T' command in sed. This is a GNU extension,
but one of the udev hotplug scripts uses it, so I need it in busybox
anyway.

Includes a test; 'svn add testsuite/sed/sed-branch-conditional-inverted'
after applying.

Doug Swarin pointed out a security bug in the -i option of sed.

While the permissions on the temp file are correct to prevent it from being
maliciously mangled by passing strangers, (created with 600, opened O_EXCL,
etc), the permissions on the _directory_ might not be, and we re-open the
file to convert the filehandle to a FILE * (and automatically get an error
message and exit if the directory's read-only or out of space or some such).

This opens a potential race condition if somebody's using dnotify on the
directory, deletes/renames the tempfile, and drops a symlink or something
there.  Somebody running sed -i as root in a world writeable directory could
do damage.

I dug up notes on an earlier discussion where we looked at the security
implications of this (unfortunately on the #uclibc channel rather than email;
I don't have a transcript, just notes-to-self) which pointed out that if the
permissions on the directory allow other people's files to be deleted/renamed
then the original file is vulnerable to sabotage anyway.  However, there are
two cases that discussion apparently didn't take into account:

1) Using another user's permissions to damage files in other directories you
can't access (standard symlink attack).

2) Reading data another user couldn't otherwise access by having the new file
belong to that other user.

This patch uses fdopen to convert the filehandle into a FILE *, rather than
reopening the file.

use more busybox functions and remove redundant code

make sure we add the local dir to the include path

Tito pointed out I'd broken -t (argv[optind] can't be before getulflags),
and replaced the use of system() (and resulting security implications).

Add automatic umount support to eject command.  Patch from Tito,
with tweaks from Mike Frysinger and Rob Landley.

Note: this will still fail to umount a path that contains an ' or \ character.
Is it worth the extra size to filter for that?

as Rob Landley pointed out, need to fix the 1 versus i typo in indexing

blah, forgot to save last time to fix whitespacing

remove duplicated code and rework algorithms to end up with smaller code

use a bunch of if statements since it is a few bytes smaller than a switch; also use bb_xfopen() instead of fopen() so comm doesnt segfault when given non-existant files :(

err, added 2 to the wrong var :) also touchup the option detection to shrink size

fix segfault if user only specifies 1 file

update usage to match actual behavior

fix whitespace/coding style

add missing comm.o rule

whitespace touchups

This makefile was a mess.  Fix it to work as intended.
 -Erik

Small comm implementatin from Rob Sullivan.  Needed to build perl.

import ether-wake applet by haveaniceday Bug 252

change the hardcoded error constant (0x80000000UL) to a nice flexible define (BB_GETOPT_ERROR)

import eject by Peter Willis / Tito Ragusa

now that mjn3 explained it to me, add documentation for the 3rd field in long opts

Patch from Jason Schoon to add IPV6 support to telnetd.  Reworked by Rob
Landley to remove an #ifdef and move another one out of the flow of code.

add documentation for long options and touch up the current docs now that i actually understand how to use the function myself :)

import more libs to prep for new e2fsprogs

import ext2fs lib to prep for new e2fsprogs

no longer applies

import base_device.c for fsck

remove unused files

trim out useless defines and use some busybox funcs

override nls P_() macro

abort if user passes -r or if they dont pass anything

syntax/whitespace touchup

Patch from Shaun Jackman:

> This patch modfies expr to use portable POSIX regex rather than BSD
> regex.
...
> This updated patch implements an anchored regex by checking that the
> match starts at offset 0.

More to the point, this patch uses the same regex that sed.c is already using
(opportunity to suck in less library code), and even building a dynamically
linked busybox with just expr the result is a slightly smaller binary (by 94
bytes, I dunno what nm --size-sort has to say about it because I didn't build
with debug info, since that changes the binary size a lot by disabling
optimization...)

Your mileage may vary.  Handle with caution.  Do not taunt happy fun ball.

Manuel points out that if printf needs a flush to act like dprintf, the result
is bigger.  Revert last patch.

This one's from me.  Fix ash "standalone shell".

If we exec /proc/self/exe and only fall back to /bin/busybox if /proc isn't
there, then we have a reasonable chance of having the standalone shell work
even if busybox isn't installed in /bin on the system in question.

Still won't work in a chroot environment, but it's an improvement.

Shaun Jackman pointed out that dprintf(STDOUT_FILENO,...) is just a printf.

patch by Tito which uses a lot more busybox functions to reduce size nicely

update e2p target to match condensed files

patch by Tito which unifies common get/set functions into 1 get/set function and cuts down on the size used significantly :)

add/remove defines to handle more e2fsprogs

use shared busybox error messages to save a few bytes

make the exec (-e) an optional feature of netcat

In bug 247, haveaniceday writes:

The option "-w secs" adds a timeout for writing.

Tweak the "pretty lsmod for 2.6" patch to be seperately selectable.

Patch from Takeharu Kato.

Revert the uptime() removal.  Let the list sort it out...

Patch from Shaun Jackman:

Replace uptime with time(NULL). time is more portable than uptime and
eliminates the need to define uptime, reducing code size slightly.

Takeharu Kato's patch added 2.6 support to lsmod; this changes menuconfig
dependencies so 2.6 support depends on insmod or lsmod...

Takeharu Kato said:

I found that lsmod in busybox does not support linux-2.6.
I fix this issue(it is caused by changes of /proc/modules format).
If you use lsmod in busybox with kernel-2.6, please use this patch.

From: Shaun Jackman <sjackman@gmail.com>
To: busybox@mail.codepoet.org

Wrap sockaddr_in6 in a ifdef CONFIG_FEATURE_IPV6.
Include sys/types.h instead of asm/types.h.
Include netinet/if_ether.h instead of linux/if_ether.h
The ioctl request argument is an int, not an unsigned short.

Shaun Jackman said:

This patch adds a CONFIG_FEATURE_CLEAN_UP stanza, and also adds an
ifdef around the SIOCGIFMAP call.

A patch from Takeharu KATO to update/fix SE-Linux support.

Add David Brownell as zcip maintainer.

Revert Tito's patch to zcip.  My bad, David Brownell had objected and I missed
it...

On Tuesday 19 April 2005 21:10, Tito wrote and today added:
> Hi,
> this is a first attempt of size optimization for zcip taking into account all
> the hints given so far on the list.
> I've applied just the more obvious busyboxifications so maybe it could be
> optimized more.
BTW: I've ripped out  a lot of debug code and changed c++ // comments to /* */
as both were rather confusing for a newbie like me. ;-)
Sorry to the author for that.
I know that this makes mantaining the code easier, but I'm simple minded....

On Wednesday 13 April 2005 09:12 pm, Shaun Jackman wrote:
> This patch fixes a memory leak in hash_file by using the BUFFER macros
> instead of xmalloc. Please apply.

David Brownell submitted a new applet, zcip, based on RFC 3927.  This is
version 0418b.  It compiled.

New help text for Matthew S. Wood's "ln" update adding -b and -d.

Patch from Matthew S. Wood:
> The following patch adds support for the -S and -b flags to `ln'.  These
> flags [especially -b] are used extensively in Debian pre and post
> installation scripts.

Comments from Vladimir Oleynik influenced the final patch, and I also ripped
out the in-file changelog since it belongs here.  At the time, it said:

/* Apr 15, 2004      Matthew S. Wood    (mwood@realmsys.com)
 *
 * Implement '-b' (backup) flag.
 * Implement '-S' (backup suffix) flag.
 *
 *
 * Mar 16, 2003      Manuel Novoa III   (mjn3@codepoet.org)
 *
 * Fixed bug involving -n option.  Essentially, -n was always in effect.
 */

On Wednesday 27 April 2005 05:23 am, Patrick Huesmann wrote:
> This is really confusing for users who don't know all details of the
> *nix file permission system.
> Today, I changed my busybox system to use CONFIG_FEATURE_SUID_CONFIG.
> After fighting with the file permissions for hours, I decided that
> updating the documentation might be a good idea.

Do not attempt to free() the application's environment, which is was
not dynamically allocated.  Instead, use a private variable to store
the environment array, which is used when we exec applications.

Correct errors preventing busybox tar from working properly,
fixing bug http://bugs.uclibc.org/view.php?id=231

fix stupid build mistake i made earlier as pointed out by psm

use a generic error message

use asprintf in place of malloc/sprintf as suggested by solar

moved to e2fsprogs too

bbify to shrink size

use busybox funcs to make smaller

new version of ed taken from sash

svn didnt auto add these ...

add new subdir for e2fsprogs

rework human_fstype to cut down size

rework options to get rid of extra variables

fix whitespace usage, fix un-escaped quotes in a bunch of examples, and standardize a bunch of trivial/full usage statements (no trailing newline in either, and when listing options, dont append a . each time)

fix inconsistent whitespace

remove extra whitespace

stat implementation based upon coreutils

patch for a very alpha busybox ed

whitespace fix

remove lsattr/chattr to prepare for a top level e2fsprogs dir with more stuff in it

update example to use bb_ prefixed names and remove old note about config.h (since the header is generated automatically now)

add comments about ignoring some warnings which are OK

remove unused variable and declare writepid inline

cast the time pointer so gcc stops warning over nothing

use bb_fflush_stdout_and_exit instead of exit

import lsattr and chattr from e2fsprogs

output busybox help to stdout, not stderr

printenv / sum defines