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.

Patch from Kent Robotti adding a bunch of needed docs!
 -Erik

Add the fdformat utility, based on a patch from Kent Robotti,
but I then completely reworked the fdformat utility to comply
with the current busybox way of doing things.

Glenn McGrath (bug1) isaway from my email till the 17th, but caught me on IRC.
He took a look into the recent reports of tar problems, and found an obvious
typo in last_patch91 from vodz which converted tar to use bb_getopt_ulflags.

Patch from Vladimir N. Oleynik (vodz):

    Last patch have changed for "ipcalc" applet with usage new get_ulflags()
    function.

Patch from Lars Kellogg-Stedman:

    Erik, et al.

    The attached patch makes the following changes to networking/ifupdown.c:

    (1) It swaps all calls to 'ip link set' and 'ip addr set'.  This solves
    two problems:

      (a) Calling 'ip link set <dev> up' before assigning an address
      generates an error message, and

      (b) Under User Mode Linux, running in with ethernet interfaces
      in daemon mode, the MAC address for an interface is selected
      based on the IP address assigned to that interface.  If the
      interface is brought up before being assigned an IP address,
      it gets a null MAC.

    (2) It further cleans up run_mapping().

    This patch is against ifupdown.c revision 1.25.

    -- Lars

Fix stupid unterminated parenthesis

Patch from Russell Coker:

    I've attached my latest SE Linux patch for busybox against the latest CVS
    version of busybox.

Patch from Kent Robotti updating fdisk to version v2.11z

Patch from Kent Robotti adding a bunch of menuconfig help

Using safe_read seems, well, safer...

As implemented, sha1sum would sometimes give the wrong answer.
This fixes it and uses faster sha1 code from Dr. Gladman.
 -Erik

Patch from Pavel Roskin to fixup toplevel help text

Apply last_patch93 from vodz:

    andersen@busybox.net wrote:
    >Message: 4
    >Modified Files:
    >       init.c
    >Log Message:
    >Remove code for unsupported kernel versions

    Hmm. Current init.c have check >= 2.2.0 kernel one time too.
    Ok. Last patch removed this point and move common init code to new file for
    /init dir

Remove bdflush garbage, which is only relevant to unsupported
kernel versions

oops.  make this actually work as intended....

Make sure we end up with a unix2dos link to busybox

Oops.

last_patch91 from vodz to convert tar to use bb_getopt_ulflags

Geir Thomassen wrote, regarding networking/httpd.c line 1358

    Hello, I think the test for an unconfigured httpd is wrong in
    the CVS (busybox-unstable-20030620.tar.bz2)

    flg_deny_all is default 0

vodz then wrote:

    Oops. You are right.
    Also, this mistake haved from two place.
    Last patch rewroted to my new get_ularg() function for overcompensate size
    from this error found ;-)

Per patch from Nick Fedchik, use SHADOW_FILE, not "/etc/shadow".
Change fopen to bb_xfopen

Oops.  As Andrew Dennison just noticed, I left a stray
space in the Makefile, thereby totally breaking it.

Regenerate the busybox.links file when the .config changes

New applet: patch, applies a unified diff