search for: inlv

Commit-ID: 43f4d1915fc244522d3fabb0fd91d23994bd472e Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=43f4d1915fc244522d3fabb0fd91d23994bd472e Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Wed, 7 Mar 2018 13:03:26 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Fri, 25 Jan 2019 02:57:21 +0000 [klibc] mystring: fix "Illegal number" on FreeBSD & macOS for x=; echo $((x)) Op 07-03-18 om 06:26 schreef Herbert Xu: > Martijn...

Commit-ID: 3f6667917c6cc2c90803e1a3e865f2c3b8b1bbf3 Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=3f6667917c6cc2c90803e1a3e865f2c3b8b1bbf3 Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Wed, 7 Mar 2018 13:03:26 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Sat, 28 Mar 2020 21:42:54 +0000 [klibc] dash: mystring: fix "Illegal number" on FreeBSD & macOS for x=; echo $((x)) [ dash commit 2b3fb53c6940471955631353b8bdb7d5a...

...not set $ src/dash -c 'foo=bar; echo ${foo=BUG}; echo $foo' barBUG bar $ This patch fixes them by introducing a new discard variable that tracks whether the extra word should be discarded or not when it is parsed. Reported-by: Martijn Dekker <martijn at inlv.org> Fixes: 3cd538634f71 ("expand: Do not reprocess data when...") Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <...

...Hutchings <ben at decadent.org.uk> CommitDate: Fri, 25 Jan 2019 02:57:21 +0000 [klibc] builtin: Reset t_wp_op in testcmd The global variable t_wp_op needs to be reset every time testcmd is called or it may cause incorrect parsing of the arguments. Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/bltin/test.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/usr/dash/bltin/test.c b/usr/dash/bltin/test.c index bab9a1f9..58c05fec 100644 --- a/...

Commit-ID: a311e6839f6e8e9797d0a0bd8e1222d22e10a018 Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=a311e6839f6e8e9797d0a0bd8e1222d22e10a018 Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Tue, 6 Mar 2018 17:40:37 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Fri, 25 Jan 2019 02:57:21 +0000 [klibc] expand: 'nolog' and 'debug' options cause "$-" to wreak havoc Op 29-03-17 om 20:02 schreef Martijn Dekker: >...

Commit-ID: bae97a14a3dab910cd57c1d36003b18a869f788f Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=bae97a14a3dab910cd57c1d36003b18a869f788f Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Wed, 7 Mar 2018 17:32:29 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Fri, 25 Jan 2019 02:57:21 +0000 [klibc] builtin: Greater resolution in test -nt / test -ot Op 07-03-18 om 15:46 schreef Martijn Dekker: > Op 06-03-18 om 09:19 schreef Herbert...

...:57:21 +0000 [klibc] eval: Restore input files in evalcommand When evalcommand invokes a command that modifies parsefile and then bails out without popping the file, we need to ensure the input file is restored so that the shell can continue to execute. Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/eval.c | 3 +++ usr/dash/input.c | 10 ++++++++-- usr/dash/input.h | 1 + 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/usr/dash/e...

...2020 21:42:54 +0000 [klibc] dash: builtin: Reset t_wp_op in testcmd [ dash commit 2e5842258bd5b252ffdaa630db09c9a19a9717ca ] The global variable t_wp_op needs to be reset every time testcmd is called or it may cause incorrect parsing of the arguments. Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/bltin/test.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/usr/dash/bltin/test.c b/usr/dash/bltin/test.c index bab9a1f9..58c05fec 100644 --- a/...

Commit-ID: 8d7c846f252b7eabd7cb7d02e7b53fb5a835402e Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=8d7c846f252b7eabd7cb7d02e7b53fb5a835402e Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Tue, 6 Mar 2018 17:40:37 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Sat, 28 Mar 2020 21:42:54 +0000 [klibc] dash: expand: 'nolog' and 'debug' options cause "$-" to wreak havoc [ dash commit 81a501b7fbcff8da004251a85a8f9...

...LENGTH expansion, the closing brace is currently not removed in evalvar. This causes the caller argstr to terminate prematurely as it would interpret the closing brace as one that belongs to a parameter expansion at the outer level. This patch fixes it. Reported-by: Martijn Dekker <martijn at inlv.org> Fixes: 3cd538634f71 ("expand: Do not reprocess data when...") Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/expand.c | 1 + 1 file changed, 1 insertion(+) diff --git a/usr/dash/expand.c...

...: Sat, 28 Mar 2020 21:42:55 +0000 [klibc] dash: expand: Fix double-decrement in argstr [ dash commit 86a841bb444ed1d9a09afb38cb818a21c04e1beb ] Due to a double decrement in argstr we may miss field separators at the end of a word in certain situations. Reported-by: Martijn Dekker <martijn at inlv.org> Fixes: 3cd538634f71 ("expand: Do not reprocess data when...") Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/expand.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/us...

Commit-ID: e86e3a7edc8934dc6a9ecd4bb360d19672f65ccf Gitweb: http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=e86e3a7edc8934dc6a9ecd4bb360d19672f65ccf Author: Martijn Dekker <martijn at inlv.org> AuthorDate: Wed, 7 Mar 2018 17:32:29 +0000 Committer: Ben Hutchings <ben at decadent.org.uk> CommitDate: Sat, 28 Mar 2020 21:42:54 +0000 [klibc] dash: builtin: Greater resolution in test -nt / test -ot [ dash commit fc914153d433b063b9ed11f3ace668c5c866f379 ] Op 07-03-18 om 15:46 s...

...mand [ dash commit 46d5a7fcea81b489819f753451c1ad2fe435f148 ] When evalcommand invokes a command that modifies parsefile and then bails out without popping the file, we need to ensure the input file is restored so that the shell can continue to execute. Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/eval.c | 3 +++ usr/dash/input.c | 10 ++++++++-- usr/dash/input.h | 1 + 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/usr/dash/e...

...leaves an open file descriptor behind. This patch fixes this by replacing REALLY_CLOSED with closed_redirs to track the current status of redirected file descriptors and leaving redirlist to only handle the previous state of redirected file descriptors. Reported-by: Martijn Dekker <martijn at inlv.org> Fixes: ce0f1900d869 ("[REDIR] Fix redirect restore on saved file...") Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/redir.c | 40 ++++++++++++++++++++++++++++++---------- 1 file changed,...

...60e502f876c36d894ba0cc2d0682b3 ] As it is a subshell can execute code that is only meant for the parent shell when it executes a longjmp that is caught by something like evalcommand. This patch fixes it by resetting the handler when entering a subshell. Reported-by: Martijn Dekker <martijn at inlv.org> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/eval.c | 4 ++++ usr/dash/main.c | 11 ++++++++--- usr/dash/main.h | 1 + 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/usr/dash/ev...

...s on exit/return [ dash commit 62cf6955f8abe875752d7163f6f3adbc7e49ebae ] We unconditionally restore the saved status in exitreset, which is incorrect as we only want to do it for exitcmd and returncmd. This patch fixes the problem by introducing EXEND. Reported-by: Martijn Dekker <martijn at inlv.org> Fixes: da30b4b78769 ("[BUILTIN] Exit without arguments in a trap...") Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/error.h | 3 ++- usr/dash/eval.c | 9 +++++---- usr/dash/exec.c | 2 +...

...@ when it's inside double quotes. In fact there is another bug while we're playing with $@/$*. When IFS is set to a non-whitespace character such as :, $* outside quotes won't remove empty fields as it should. This patch fixes both problems. Reported-by: Martijn Dekker <martijn at inlv.org> Suggested-by: Harald van Dijk <harald at gigawatt.nl> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/expand.c | 38 ++++++++++++++++++++++++++------------ 1 file changed, 26 insertions(+), 12...

...@ when it's inside double quotes. In fact there is another bug while we're playing with $@/$*. When IFS is set to a non-whitespace character such as :, $* outside quotes won't remove empty fields as it should. This patch fixes both problems. Reported-by: Martijn Dekker <martijn at inlv.org> Suggested-by: Harald van Dijk <harald at gigawatt.nl> Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au> Signed-off-by: Ben Hutchings <ben at decadent.org.uk> --- usr/dash/expand.c | 38 ++++++++++++++++++++++++++------------ 1 file changed, 26 insertions(+), 12...