Mark Lord writes:

The config file processing for inetd in busybox-1.00-pre7 and newer
is broken -- the saved command argv[] values overwrite each other.

Patch to fix it is attached.

Cheers
--
Mark Lord
Real-Time Remedies Inc.

Add the -r option, patch from Rob with some help from myself.

Remove debugging

Dont strip trailing '/' until _after_ i test to set if its there !

Patch from Florian Schirmer <jolt@tuxbox.org>.  When I rewrote the parser,
I overlooked the case of someone wanting to use a valid but empty suid
config file.

Woops, im getting ahead of myself, we dont have the -r (refernece)
option yet

define option names to be clearer, simplify nested if statements, remove
un-needed if statement, minor indenting change

Avoid naming conflict with symbol in newer glibc headers

Accomodate the fact that newer libc versions may in fact not
contain query_module and friends and more

oops.  Leave the default feature set enbled for now...

Initial effort at disabling job control as well

Support disabling pipe and redirect support

Update URL

Buffalo appears to have resolved their GPL problems

Richard Kojedzinszky writes:

Hi All,

I aplogoize for the mistake, but i have just recognized that somehow the
last patch I sent in was wrong, and a '0' was instead of a '-1'. Because
of this, vi does behave the wrong way. So again, it should be the last
patch for vi. This is for pre7.

Patch from vodz, based on bug report from Richard Kojedzinszky:

Richard,

>I have a problem, which I can reproduce now. I am using pre7 version of
>busybox, and the tab completion works fine. I mean, with an empty command
>line I press the TAB twice, and ash shows me the available commands. But
>when i process the profile file below, as
> $ . /etc/profile
>then it stops working, and the double-tab lists the directories available
>from the cwd, and not the commands. Has someone else meet this problem
>before, or am i doing something wrong?
>
>This is  my '/etc/profile':
>- ---
># System profile
>
>PATH=/bin:/sbin:/usr/bin:/usr/sbin:/usr/local/bin:/usr/local/sbin
>export PATH
>trap ":" INT QUIT TERM
>
>export PS1="\h \w # "

Thanks. Patch attached.

--w
vodz

Make the loop support stuff be much less evil, and make it cope
with 2.6.x asm/posix_types.h, which has done singularly evil thing
by yanking __kernel_dev_t and renaming it.  The loop interface was
really poorly designed in the first place.  The new 64 bit loop
interface looks to be somewhat less horrible, too bad it is only
present in 2.6.x kernels.
 -Erik

Eliminate use of a kernel scsi header file.

Prevent potentially misaligned accesses while indexing a pointer
to the partition table, which would be a bad thing on i.e. arm.

Make sure stdlib.h is always included before dmalloc.h to avoid problems
parsing problems.

Vladimir N. Oleynik writes:

Hi, Glenn.

Current CVS "ls" applet have small problem: some options
ignoring. Last patch attached ;-)

--w
vodz

Vladimir N. Oleynik writes:

Ok. I found my mistake :(
The trivial patch attached.

--w
vodz

Elaborate on CONFIG_SELINUX, and explain that most people should
not enbable this option.

Grammar fix

Minor grammar fixup

Bump version number for release

Richard Kojedzinszky writes:

Hi,

I've noticed the bug also, and here is another patch for it. I hope it'll
not introduce more bugs. Not too nice, but works for me.

Here it is for busybox-1.00-pre6

Joe.C writes:

Hi,

   When httpd connection is closed, bosybox httpd will
not stop reading from CGI program. This patch fix this
problem. It check the return value of bb_full_write and
stop reading from CGI if the connection is closed.
Please apply this patch.

Joe.C

Jean Wolter writes:

Hello,

when calling seq with

    seq 1 1

it generates an "endless" list of numbers until the counter wraps and
reaches 1 again. The follwoing small patch should introduce the
expected behavior (output of 1 and termination):

regards,
Jean

Rob Landley writes:

While building glibc with busybox as part of the development environment, I
found a bug in glibc's regexec can throw sed into an endless loop.  This
fixes it.  Should I put an #ifdef around it or something?  (Note, this patch
also contains the "this is not gnu sed 4.0" hack I posted earlier, which is
also needed to build glibc...)

The variable 'complementaly' used to be allocated with calloc, which zeroed it
out during the allocation process.  When vodz changed it to be allocated on the
stack, he forgot to explicitly zero it, leaving its value filled with whatever
used to be sitting on the stack.  It would garbage values, depending on the
garbage that happened to be sitting on the stack when the function was called.
The result was that applets using bb_getopt_ulflags() were showing
unpredictable behavior (such as segfaults), which naturally broke many things.

Bug fix from Tito.

Vodz last_patch_125_2, this patch have:
- synced with dash 0.4.21
- better handle trap "cmds..." SIGINT (strange, i make bad hack for ash
  and cmdedit, but this work only with this...)
- may be haven`t problem with Ctrl-D

Rewrite parse_config_file().  Among the old version's problems:
  No checking for lines that were too long.
  No checking that fgets returning NULL was actually due to EOF.
  Various whitespace handling inconsistencies.
  Bloat (switches and multiple identical function calls).
  Failure to check for trailing characters in some cases.
  Dynamicly allocated memory was not free()d on error.
Given that this controls suid/sgid behavior, the sloppy coding was
really inexcusable.  :-(

Use bb_xstrdup() instead of strdup().

Fix http proxy use, bytes were swapped

Mention changes to the daily snapshots

Update docs for release

Bump version for release

Use standard C99 types

Use proper C99 types

Fixup use of 'u_int' to instead use 'unsigned int'

s/u_int/uint/g

s/u_int/unsigned int/g
s/u_short/unsigned short/g

Use proper C99 types

Use proper C99 types throughout.  So not use silly typedefs.

Avoid symbol naming conflict with libm

Support new uClibc stdio core.

Apply a couple of optimizations

Teach busybox ps to get the correct uid when displaying stuff

New applet, seq. No options, just the basics.

Align using spaces to furthurest character and then one tab, now they
should always be aligned.

A few little updates, mostly cosmetic

Sigh.  TRENDware has released source -- only for busybox and udhcp.
So despite their not providing kernel source, they have compiled with
the requirements for busybox at least...

Pascal Brisset writes:

uuencode fails to encode binary data because it right-shifts
bytes as signed chars and keeps the duplicated sign bits.

The original base64_encode() from wget/http.c is broken as well,
but it is only used to encode ascii data.

-- Pascal

Be stricter when converting strings to integers.  Should fix the problem
reported by Rob.

Fix compile error when tab completion disabled

Add the -r option as a synonym of -R

Add the -h, --dereference option for archive creation.

Enable long options, adds 150 bytes.

Correct check for only one of K or S options

Check valid options were given

Dont change ownership unless we created the directory.

Remove unneeded conditions and logic, fix bug where ownership didnt
change.
Dont use bb_make_directory it doesnt have the features, and its ugly to
work around it.

last_patch_124 from Vodz, fix an ash bug when alt-1 was pressed, debian
bug #228915

Check one and only one of start, stop are given.
Remove some global variables.
#define some getopt values.

Vodz, last_patch_123, patch have new version getopt_ulflags.
- size reduced 34 bytes
- don`t use dynamic memory allocation
- small indent correction.

Patch from Atsushi Nemoto, recent MIPS kernel headers
does not provide PAGE_SHIFT for userland (because now mips-linux kernel
supports PAGESIZE other than 4K).

Patch by Richard Kojedzinszky, when using END at end of lines it was
 skipping to next line, cw command was leaving one char in buffer

Redo getconfigent, save 150 bytes, still small memory leak when
parsing invalid entries.

Patch from Tito, reduce size, merge functions that are only used once.

Stephane Billiart writes:

bb_lookup_port now takes 3 parameters but rdate has not been modified
accordingly and fails to compile in the current CVS version.
The modification below fixes the problem.

Now, RFC868 allows both UDP and TCP implementations of the time protocol
so this may not work if someone defines a udp time service other than 37
but who would do that?

Fix up a couple of bugs i introduced in yesterdays cleanup.

Fix a bug where `ls -le` would print the time twice.

Use bb_getopt_ulflags, save 200-300 bytes

Patch from Leif Lindholm (slightly modified by me), to fix http
redirection.
Example of incorrect behaviour was the following url, busybox wget didnt
get redirected to the busybox url as it should.
http://freshmeat.net/redir/busybox/953/url_homepage/www.busybox.net

Modify bb_lookup_port to allow the protocol to be specified, allowing
/etc/services support for inetd, netcat and tftp.

Patch from Arthur Othieno, clarify common options between dpkg and dpkg-deb

The functions setconfig, enter and bump_nofile were only called once, marge them into the calling
function.

Use 1 line for function name and return value,
remove dofork define/variable,
dont check pid value is < or > than 0 if we know it is 0.

Remove some defined statements

Changed #define names for common features (superficial)

Use bb_get_chomped_line_from_file

Minor updates from linux 2.6.1

Patch by Tito, reduce size of deallocvt, also make the usage message
clearer (from N to [N]) and indirectly (through bb_xgetlarg) adds support
for  -h ;-)

Vodz last_patch_122, Check $PATH at runtime to fix tab completion

Patch by Tito, use bb_xgetlarg, reduce size

Fix a bug where mount could check the wrong device.  st_rdev is the correct
device ID iff the named file is a character or block special device.  Otherwise
it is meaningless junk, in which case st_dev should be used.  This was done
incorrectly, which could cause mount to display bogus mount info.
 -Erik

Patch from Bastian Blank, allow the buildtime to be overriden with a
different string, used by the debian busybox-cvs to specify the debian
date based version number.

Vodz last_patch_121, syncing with dash_0.4.19, reduce code size.

Patch by Tito, remove some unneeded variables to save some space.

Patch from Matt Kraai to fix debian bug #227081

cp does not truncate existing destinations.  That is, after
running

 echo foo > foo
 echo fubar > fubar
 cp foo fubar

the contents of fubar are

 foo
 r

instead of

 foo

whitespace cleanup

Woody Suwalski writes:

I have found the problem in modprobe, so here is the promised patch
At the current stage I can use it as modprobe while switching between
2.4 and 2.6 seemlesly...(that is good!)

Patch from Bastian Blank to fix debian bug #226722, test case:
 mkdir foo
 touch foo/bar
 ln -s bar foo/baz

Use bb_getopt_ulflags, save 100 bytes and strict argument checking.

Woody Suwalski writes:

accept more then 1 dependency per modules.dep line. Also white space cleanup...
I think that parsing still breaks sometimes, but is mostly functional now.

Fix broken #else

Use bb_getopt_ulflags, save 150 bytes.

Use bb_getopt_ulflags, simplify some logic, saves some bytes.

Fix usage with libraries, skip symbol table.