Joe.C writes:

    I've reported this bug in April and it still
    exists in 1.00-pre2. So I made patches for
    both 0.60.x and 1.00-pre2.  The patch is very
    simple, just use strncmp instead of strcmp.
    Please apply if it is OK.

   Here's the procedure to test this problem:
   Create a executable with very long name, say
   'test_1234567890123456' and execute it. Try
   using 'killall' or 'pidof' to find/kill this
   program.  Without this patch, you can't find
   the program.

Fixup segfault on 'kill -q'

extern inline is a bad bad thing.  kill it (so mkfs_minix.c will
actually compile)

Seems I missed a spot

Merge/rework config system per the latest from linux-2.6.0-test2.
Fix the config bugs revealed by the updated config system.
 -Erik

Might as well get my name in AUTHORS before 1.0...

Save 0.5K.

Remove code that too agressively disabled inlines

Minor cleanup

Fix bug when handling debian packages.
If we read the contents of compressed files within the ar archive,
e.g. control.tar.gz, then file position gets all out of whack, so
it has to be reset before reading thenext header.

Remove LRP (now defunct) and replace it with leaf.

Update webpage and changelog for release

Lars Ekman writes:

When using "losetup" the device is always setup as Read-Only.

(I have only tested with the -o flag, but looking at the code the
 problem seems general)

The problem is the "opt" variable in "losetup.c" that is reused in
the "set_loop()" call. Clear it before the call and everything is OK;

  opt = 0;         /* <-------- added line */
  if (delete)
    return del_loop (argv[optind]) ? EXIT_SUCCESS : EXIT_FAILURE;
  else
    return set_loop (argv[optind], argv[optind + 1], offset, &opt)
      ? EXIT_FAILURE : EXIT_SUCCESS;
}

Best Regards,
Lars Ekman

last_patch100 from vidz updating fdisk to 2.12pre

Do not require that the signal number be specified

Eric Spakman writes:

The recent changes in ifupdown where all calls to 'ip link set' and
'ip addr set' are swapped give some problems with v4tunnels.
For plain ipv4 and ipv6 interfaces it works correct, other methods
not tried. The patch below change the behaviour back for v4tunnels
only.

Without the patch the following errors are shown:
RTNETLINK answers: Network is down
RTNETLINK answers: No route to host
and the tunnel is not fully brought up

With this patch all works as expected.

Vladimir N. Oleynik writes:

    This moment have algoritmicaly problem, not overflow:
    strcat(wrapped, wrapped) - may be looped.

    Hand patch:

    - else if (strstr(strcat(wrapped, wrapped), newmono))
    + else {
    +    safe_strncpy(wrapped + lenwrap, wrapped, lenwrap + 1);
    +    if (strstr(wrapped, newmono))
    +}

    --w
    vodz

Christian Meyer provided this patch to fix more bugs with the tftp client

Ronny L Nilsson writes:

The login process should always timeout if user don't login sucessfully within
reasonable time. Otherwise we're sensetive to a DOS attack by simply doing a
bunch of simultaneous telnet connections (deploys all availible TTY's).

This patch make login.c terminate the connection after  "TIMEOUT" seconds.

Ronny L Nilsson writes:

    If BusyBox was compiled with -DCONFIG_FEATURE_CLEAN_UP dmesg command
    segfaults if invoked with the "-n" option. (Due to a free() of an
    uninitialized pointer).

Set libbb dead last in the link order

Match up interfaces

Fixup typo noticed by Nick Fedchik

Bruno Randolf writes:

this patch fixes run_parts when it's called by ifupdown. 1) argv has to be a
NULL terminated char* array, not just a string. 2) run_parts now explicitly
sets the environment. this environment is populated from the
/etc/network/interfaces config file and is needed by the scripts in
/etc/network/if-pre-up.d/. when run-parts is called from the command line the
environment is taken from the current process.

Vladimir Oleynik then wrote:

You can simplify this if use:

+       bb_xasprintf(&buf[0], "/etc/network/if-%s.d", opt);
+       buf[1] = NULL;
+
+       run_parts(&buf, 2, environ);
+       free(buf[0]);

--w
vodz

Vladimir N. Oleynik writes:

Last patch have synced form Manuel Nova III xxreadtoken() function,
corrected (C) form dash debian/copyright, removed my small mistake
with IFS_BROKEN (thanks by Herbert), and synced cmdedit.c from
current CVS (removed libc5 support, your email correction, my (C) year
corertion).

Yang Xiaopeng writes:

>I'm sure that no user process use old root now,  but when run "umount
>/old_root", it says:
>   umount: /old_root: Device or resource busy
>
>I have tried to remount /proc within the new root *after* chroot, but
>get the same result.
>
>
I found the problem, I said that no user process use old root when run
my scripts, but
I'm wrong, actually there is a '3' fd open the file
"/old_root/dev/console". By adding
debug message in init/init.c, I found the problem: when init restart(in
exec_signal()),
before open the new terminal device, there is still a file opened(I
don't know which file it is), so the
terminal device(stdin) get fd '1', and the first dup(0)(stdout) return
'2', the second(stderr) return '3'.

I attach a simple patch to solve this problem.

bump version to 1.0.0-pre2

A small update to the new docs.  Plenty more is needed...

Patch from vodz to update httpd usage

This is synced from dash-0.4.17 and full ready for insert to new busybox
version:
ftp://ftp.simtreas.ru/pub/my/bb/new

News:

- code is smalest!
- support ${var...} expr
- used new very strongly steal controlling terminal

Avoid shadowing built-in function `log'

Fixup bugs in last patch

Update docs a bit

last_patch98 from vodz:

Denis,

># ./busybox env - echo zzz
>zzz
># ./busybox echo -n zzz
>zzz
># ./busybox env - echo -n zzz
>env: invalid option -- n
>
>       obviously, env tried to understand -n as env's option
>       instead of blindly passing it to echo...
>
>BusyBox v1.00-pre1 (2003.07.16-07:53+0000) multi-call binary
>
>Usage: env [-iu] [-] [name=value]... [command]

Ah, you found very old problem.
Last patch also have:

- multiple "-u unsetenv" support
- GNU long option support
- save errno after exec failed before bb_perror_msg()

--w
vodz

last_patch95 from vodz:

Hi.

Last patch have new libbb function
vfork_rexec() for can use daemon() to uClinux system.
This patched daemons: syslog, klogd, inetd, crond.
This not tested! I havn`t this systems.
Also. Previous patch for feature request MD5 crypt password for
httpd don`t sended to this mailist on 07/15/03
(mailist have Pytom module problem?).
The previous patch included, and have testing.

--w
vodz

Add a "search google" button

James Petterson writes:

I've found a possible bug in libbb/interface.c, in function
if_readlist_proc(). This function calls get_name(), and passes
as an argument 'name', a buffer of 16 bytes (IFNAMSIZ).  The
function get_name(), however, may use more than 16 bytes,
when it is searching for aliases. Even if you don't have an
alias interface, you can run into trouble if the interface
has received more than 99999999 bytes, in which case the
space between the interface name and the rx stats
disappears, as in the /proc/net/dev example below:
 wan0.200:264573315  462080    ...
In this case get_name() correctly identifies the interface name
as "wan0.200", but to do that it uses 18 bytes of the 'name'
buffer, which could lead to an unpredictable error.

A simple solution would be to increase the size of the buffer:

There should be only one instance of CONFIG_FEATURE_AUTOWIDTH

Allow people to use the full blown iproute2 programs with busybox ifupdown.

Move start_stop_daemon to debianutils.
Cleanup run_parts a bit and add long opts

cleanup and add long options

The logger utility does not really depend on CONFIG_SYSLOGD,
since it can also be used with other syslog daemons
 -Erik

Patch from Christian Meyer:

    The client gives up way too soon because timeout is set to 0 ...
    There's a solution for that problem.

Some additional description based on a patch from Terje Kvernes

Patch from Nick Fedchik to fixup paths in busybox/libpwdgrp
which were not properly using the bb_path_*_file strings.

Rework kill / killall so it behaves itself, even when subjected
to abuse.
 -Erik

Patch from Thomas Gleixner to reap any zombie processes that are
reparented to init...

Don't depend on CONFIG_LFS.  It is really a suggestion, not a requirement,
since fdisk will work just fine on smaller disks w/o it.

Set the tm_isdst flag to -1 before calling mktime().  Otherwise, the current
timezone setting is used for the new date.

eliminate DOSTATIC, DODEBUG, etc and name them sensibly

Several simplifications and indenting changes, per
last_patch97 from vodz

As vodz noticed, I screwed up reboot

the word "only" is repeated for CONFIG_FEATURE_SHADOWPASSWDS

There is no need to expose CONFIG_FEATURE_TRACEROUTE_SO_DEBUG

Document CONFIG_FEATURE_INSMOD_LOADINKMEM

Document CONFIG_ASH_CMDCMD

Write some more docs

Remove the update utility, that is only relevant to older 2.0.x kernels
that are no longer supported.

Patch from Terje Kvernes adding quite a lot of missing documentation

Patch from  Andrew Flegg:

    Here's a pretty crude patch to reload /etc/inittab when init receives a
    SIGHUP. The mailing list archives weren't entirely clear on whether or
    not it should already happen, but didn't appear to be.

    The patch:
       * Adds a new function, reload_signal() which just calls
 parse_inittab() and run_actions(RESPAWN)

       * Before entering the while (1) loop set up SIGHUP to call
 reload_signal()

       * Modify new_init_action to skip the action if the same command
 already exists on the same terminal

    This last bit means that changing already running entries is a bit
    hairy as you can end up with, for example, two shells running on the
    same virtual console. However, for solely adding/removing entries this patch
    seems to work quite well.

Support reboot, halt, and poweroff independent of busybox init.
Simplify and fixup some logic.
 -Erik

Remove remaining libc5 support code

A minor update for handling older gcc versions

Update TODO list

Add a default clean target

Very minor rdate updates

Error out early if people try to build mount with nfs support
using uClibc, but have not enabled UCLIBC_HAS_RPC...
 -Erik

mention stresslinux

Paul Mundt <lethal@linux-sh.org> writes:

Here's a bunch of fixes for the watchdog app in busybox. This does a
couple of things:

- configurable timer duration (userspace timer duration is
  usually configurable within the device drivers themselves).
- run as a daemon
- shutdown the device properly on SIGINT or SIGHUP
- clear the counter immediately instead of sleeping first

as well as updating the usage information. This has also been switched
over to getopt to deal with the optional timer duration specifier.

The changes themselves are harmless and isolated, and I've veried that
this works on sh and x86 without any problems.

Patch from Paul Mundt:

    The current SC_x references automatically assume that everyone uses ttyS/tts
    as their naming scheme for their serial ports. This isn't the case for quite
    a few architectures, including sh, sh64, h8, arm, etc.

Matteo Croce submitted a hdparm applet.

Patch from Jeff Angielski <jeff@theptrgroup.com> recognizing
his work on ftpput and ftpget.

Fixup some little error in the webpage.  Mention the Dell TrueMobile 1184.

Update required kernel version and note that libc5 is no longer
supported (it is unmaintained and lacks support for simple things
such as stdint.h) and uClibc is a better choice these days anyways.

Fixup problem unconditionally converting all hard links to symlinks.

Update changelog and docs

Document CONFIG_USE_BB_PWD_GRP

Update the default set of applets that are listed

Yet more preparation to make a release...

Fix a silly typo

Looks like I forgot one little tiny detail...

Update a bunch of docs.  Run a script to update my email addr.

decouple delgroup and deluser

Patch from Thomas Cameron:

    Hello all,
    This patch adds more "Help" text to the config system.  Almost
    all applets now have a help entry.  Also, I cleaned up the spacing of
    the existing text so that things are consistent.  This patch is against
    this morning's CVS.

    Thomas Cameron
    CEI Systems, Inc.

Use standard types

Michael Sternberg writes:

    Latest BusyBox from CVS (Jul 7 2003).
    File coreutils/ls.c
    "if" at line 970 should be enclosed in #ifdef CONFIG_FEATURE_LS_SORTFILES
    "if" at line 976 should be enclosed in #ifdef CONFIG_FEATURE_LS_TIMESTAMPS

Patch from Lars Kellogg-Stedman:

    The attached two patches add config documentation to init/Config.in and
    fileutils/Config.in.

Patch from Lars Kellogg-Stedman:

    Busybox vi is installed in /bin/vi, but crontab was looking for
    /usr/bin/vi.  This patch makes crontab look for /bin/vi instead.

Require that LFS is enabled to get fdisk

Remove all the llseek junk and just use regular old lseek.  When DOLFS is
enabled, regular lseek is transparently promoted to lseek64 anyways, rendering
the llseek stuff pointless.
 -Erik

On second thought, this is a better way to fix things

Don't use Linux kernel headers
 -Erik

This declaration no longer needs to hide when using dmalloc

Be certain to not abort prematurely when reading stuff from pipes.

Fixup compile problem with dmalloc

As noted by Thomas Eckert:

    bb-tar "cjf" does not create a valid tbz2-archive -- if fact the result is a
    plain tar-file (no compression) -- but does not warn about the unrecognized
    parameter combination "cj" (bb does not have bzip2-compression yet, right?).

to fix this I have added an error message stating this does not work.

He also reported

    cosmetic: versose "-v" does not show any output when used with "create"

which I have now fixed as well.

 -Erik

oops

Daniel writes:

I've found a problem with job control when the init process is restarted.

If the system boots for the first time, I get job control on a serial terminal -
no problems. However, when I restart init by issuing "init -q", then the shell
no longer has job control.

I traced this a problem in console_init in the file init.c. What was happening
after the restart is that the first compare

    if (ioctl(0, TIOCGSERIAL, &sr) == 0) {
...
    } else if (ioctl(0, VT_GETSTATE, &vt) == 0) {
...
    } else {
... // assume /dev/console
    }

returned error and subsequently the code assumes /dev/console as the console,
which does not support job control.

Checking the errno after the first call showed that the system was complaining
about the file descriptor. This is probably because the previous init process
had closed all its file descriptors which the new init process had inherited.

Patch from Lars Kellogg-Stedman:

    Glibc 2.1.3 (used by the Hardhat Linux SDK distributed by Cyclades)
    does not define IF_NAMESIZE in net/if.h.

Patch from Lars Kellogg-Stedman:

    This patch fixes endian problems with get_netmask().  I don't know if
    this is the cleanest solution, but it makes 'ipcalc -n' work on both
    an i386 system and a ppc system.