2005-07-05  Shaun Jackman  <sjackman@gmail.com>

        * init/Config.in (CONFIG_FEATURE_INIT_SWAPON): New option.
        * init/init.c (check_memory): Disable the swapon feature unless
        CONFIG_FEATURE_INIT_SWAPON is defined.

rename log var to log_console so we dont override internal gcc/glibc log func

2005-06-30  Shaun Jackman  <sjackman@gmail.com>

        * loginutils/getty.c: (open_tty): Use dup2 instead of close/dup.

2005-06-30  Shaun Jackman  <sjackman@gmail.com>

        * loginutils/getty.c: Include utmp.h only if
        CONFIG_FEATURE_U_W_TMP is defined.
        (getty_main): Use ISSUE only if it is defined.

Patch by jonlar in Bug 312 to split the U_W_TMP feature into sep UTMP and WTMP options

patch by Shaun Jackman to combine dup/close funcs into dup2

dont use f_frsize unless linux-2.6.0 or better

rip out all the non-linux code and ugly workarounds

tweak signed/unsigned char usage to avoid mismatches

- remove extra/unneeded function call. testing svn

Enabling runtime SUID/SGID configuration via /etc/busybox.conf
is not a very good default.  Better to default to having it off
and let people get the default behavior.  If they want to enable
/etc/busybox.conf they should explicitly ask for it.

applets specified as _BB_SUID_ALWAYS in applets.h should also select
CONFIG_FEATURE_SUID to ensure proper behavior when installed.

characters encoded as html should have a trailing semicolon
to be interpreted properly

Rodney Radford submitted ipcs and ipcrm (system V IPC stuff).  They could use
some more work to shrink them down.

Tito says: unify verbose/quiet flags

Tito says: strip unused program_name
Vladimir N. Oleynik says: uname() can be replaced with get_kernel_revision()

use xmalloc instead of malloc

use malloc instead of xmalloc

use xmalloc() instead of malloc()

use xmalloc() and bb_perror_msg_and_die()

Note that memory allocaiton needs to be cleaned up too.

remove com_err.h includes

DOS only crap

import initial fat mke2fs

import tune2fs support

fix signed/unsigned char pointers

oops, we only want to affect local CFLAGS

initial fat tune2fs/findfs/e2label source

replace simple is_null func with a memcmp define

force including of e2fsbb.h and move the HAVE_* defines to it

move config.h requirement to the actual .depend target rather than the depend alias

make sure clean removes objects in subdirs

need strings.h for ffs()

only define some variables if legacy EXT2FS_ENABLE_SWAPFS is enabled

replace simple functions with defines

whitespace updates

replace functions with defines

whitespace updates

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.