Vodz. last_patch_128
- declare applet_using as static from applets.c
- small correction to cmdedit,
   previous version cleared history after Ctrl-C
- small spelling correction (by Friedrich Lobenstock)

Patch from James Zhu, telnetd window resizing support.

Use bb_getopt_ulflags, marginal saving, better argument checking.

Use /var/lib/hwclock for adjtime, its really a state file rather than a
config file, so it should be in /etc, FHS mentions it also.

Patch from Florian Schirmer. compile fix for debug builds

Patch from Hideki IWAMOTO, output debugging messages to stderr

remove duplicate header declarations

Fixup braces (patch from Thomas Geulig), run through indent and manually
adjust.

Thomas Geulig, remove duplicate header

Update usage for ftpget and ftpput

Ptch by Hideki IWAMOTO, fix a bug preventing tftp from getting or
putting more than 0xffff blocks.

Return 1 upon failure

No need to check proxy env variable twice, ignore proxy env variable if
its set to 0

Sometimes i get carried away with the use of function pointers, im sure
it seemed like a good idea at the time.

Use return instead of exit, use == instead of & ==, left justify labels,
adjustment of whitespace.

Document missing cp options

Fix up hard links

A strict interpretation of the ustar format requires the type flag to be
interpreted, we cannot depend on the file type being set in the mode
field.

Add extra comments, freakout if a pax archive is encoutered.

Chris Larson (kergoth) writes:

As Manuel points out, this is a flawed fix, and doesnt fix the
following:

mkdir -p cpa cpb cpc
cp -a cpa cpa/cpb/cpc

Attached what appears to be a more sane fix.  Apply on top of previous.
Please confirm sanity.

Chris Larson (kergoth) writes:

I was adding -s/--symbolic-link support to busybox cp when I noticed a
bug with -r/-a.  Test case:

mkdir -p test/out
cd test
busybox cp -a * out/

Will never return until we run out of open files or similar.
Coreutils cp on the other hand will error with "cannot copy a directory,
`out', into itself, `out'".  Patch attached.

Dont use same variable and struct name

Formatting

remove debugging

Patch from OpenWrt.

Add -i option to sed, to edit files in-place.

Fix vstype[] to match VS* defines.  Patch from OpenWrt by mbm.

Remove the unused CONFIG_FEATURE_HTTPD_SET_CGI_VARS_TO_ENV option.

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.