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

* libbb/printf.c: Check for __NEWLIB_H__ before __GLIBC__.

2005-03-19  Shaun Jackman  <sjackman@gmail.com>

* networking/ping.c (ping): Change the type of fromlen to socklen_t.

remove unused variable ret as reported by apgo in Bug 350 and touchup syntax along the way

no longer used as reported by apgo in Bug 348

just punt all the f_frsize crap since not all linux headers support it Bug 346

ifdef all of loop.c with CONFIG_FEATURE_MOUNT_LOOP.  won't
compile due to CONFIG_FEATURE_MOUNT_LOOP_MAX otherwise.  reported
by Stephane Billiart.

applying fix for:
     0000093: Patch for dpkg - can't handle scripts
      Attached patch is needed to fix dpkg's support for preinst,
       postinst etc script files.

remove duplicate check against chaddr.

revert 10881, and refix by changing "if (vallen)" to "if (val)".  this
is per the upstream fix for dash, in dash_0.5.2-6.diff.  thanks vodz, for
catching this.

applying fixes from:
    0000142: unzip enhancements

applying fix for:
    0000026: poor man's "scriptable" telnet

applying fix for:
    0000271: [PATCH] tftp -g fails if a TFTP_ACK is lost

applying fix for:
    0000265: tail -f should keep following files even if they
                    were truncated

applying fix for:
    0000263: nc cannot use -e when initiating a tcp connection
     to something else

applying fix for:
     0000262: tar -x doesn't believe it has reached the end of archive

applying fix for:
    0000261: Unsafe empty env var export in ash

applying fix for:
     0000260: udhcpc doesn't validate client hardware address

applying patch from:
    0000227: sort use wrong type for getopt return variable

applying fix for:
    0000203: 'ip route flush cache' not implemented

applying fix for:
    0000185: httpd infinite loop when piping to CGI script

applyinf fix for:
    0000155: variable expansion with braces in backticks in msh

applying fix from:
    0000152: ash: quoting rules for local variables different to globals

applying fix for:
     0000143: sysklogd remote logging adds a space between facility and tag

applying fix for:
    0000118: vi join command does not mark file as modified for certain lines.

much more concise fix for bug #45.  just align the packet...

If /tmp and /home were different partitions, then "mv /tmp/file /home/file"
would delete /home/file even if /tmp/file didn't exist.

This fixes that, although the logic of both mv and cp is a bit tangled and
should probably be untangled.

applying fix for:
     0000117: Remove linefeed after overwrite prompt using cp -i
  User input not on the same line as the prompt when about to
  overwrite a file.

applying fix from:
    0000108: busyboxy/networking/ftpgetput.c not conforming to
                    RFC 959.  ftpget and ftpput send <LF> as EOL.

applying fix from;
    0000092: looks like the initializer for .need_suid was missing.

applying fix from:
    0000088: inetd chargen stream does not generate the
    characters as recommended in RFC 864
    Chragen service is generating garbage characters.

applying fix from:
     0000068: mount limited to max 8 loop devices (patch provided)

(made minor wording change for config help message)

applying fix from:
 0000067: cp -p produces misleading error message

applying:
0000054: Tab completing filenames in ash causes SEGV
Simple tab completion operations cause busybox (ash) to
access illegal addresses.

applying fix for:
 0000045: traceroute causes an alignment trap due to unaligned buffer on arm

Fixup makedevs to handle regular files, and also fix
it to properly update file permissions as specified.

fix for "0000027: patch: nc will spin if stdin closed"

allow both ^H and DEL to backspace in insert mode (bug #23)

Fix vi so that error messages, insert mode messages, etc are
all actually displayed in the status line as expected

a bit more polish

a bit of polish on makedevs

the makedevs config option was pretty much totally broken

Fixup device table based makedevs so it actually works

move var decls around a little to help gcc make smaller code

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

        * init/init.c: Do not include sys/mount.h.
        (message): Use O_NONBLOCK instead of O_NDELAY.
        (console_init): Ditto.

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?