diff options
author | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2004-03-16 05:14:10 +0000 |
---|---|---|
committer | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2004-03-16 05:14:10 +0000 |
commit | 79ad967cd66dd37561088ed5ccd9dd65d9502c24 (patch) | |
tree | 60d41b5ef57193173d1eecd747a0ddb3130e90ea /libbb | |
parent | 41fde650025d57fc0acdaaf7bd47bd20a5b109ad (diff) | |
download | busybox-w32-79ad967cd66dd37561088ed5ccd9dd65d9502c24.tar.gz busybox-w32-79ad967cd66dd37561088ed5ccd9dd65d9502c24.tar.bz2 busybox-w32-79ad967cd66dd37561088ed5ccd9dd65d9502c24.zip |
Patch from vodz to fix the dynamic vars patch, which I should not
have checked in. Vladimir writes:
Your patch have many problem.
1. You always added + time(). This cannot reset RANDOM=value for debuging
with
replay sequential.
2. Hmm. I examine bash 2.04 source. This pseudorandom generator use low bits
of
counter value. You use high bits. This make bad pseudorandom values after
have
0-value. For example, if + time() do remove, your generator always return 0
after
first generate 0.
3. Memory leak per call. Use ash-unlike unecessary bb_strdup function.
4. Unsupport show last $RANDOM value for "set" and "export" command.
5. Bloat code. Busybox-unlike patch - added unstandart feature as default
hardcode.
Last patch attached.
Erik, why you apply Paul patch with have 5-th point problem? :(
Last patch have ash change xwrite() to fresh libbb/bb_full_write interfase
(haved loop after EINTR).
--w
vodz
git-svn-id: svn://busybox.net/trunk/busybox@8632 69ca8d6d-28ef-0310-b511-8ec308f3f277
Diffstat (limited to 'libbb')
0 files changed, 0 insertions, 0 deletions