summaryrefslogtreecommitdiff
path: root/coreutils
diff options
context:
space:
mode:
authorDenis Vlasenko2008-04-30 00:39:22 +0000
committerDenis Vlasenko2008-04-30 00:39:22 +0000
commitdcbd350ccfe7eaa4feab038f97a382684f2adea9 (patch)
treeb06da24101aa74d8dd25b1c51461bda52d675388 /coreutils
parenta2dcb5017503acea97547e4c538a47f920d752f7 (diff)
downloadbusybox-dcbd350ccfe7eaa4feab038f97a382684f2adea9.zip
busybox-dcbd350ccfe7eaa4feab038f97a382684f2adea9.tar.gz
echo: fix echo -e -n "msg\n\0"
(by "Pinedo, David" <david.pinedo AT hp.com>)
Diffstat (limited to 'coreutils')
-rw-r--r--coreutils/echo.c12
1 files changed, 8 insertions, 4 deletions
diff --git a/coreutils/echo.c b/coreutils/echo.c
index 6e25db6..cc9b9e6 100644
--- a/coreutils/echo.c
+++ b/coreutils/echo.c
@@ -108,15 +108,19 @@ int echo_main(int argc ATTRIBUTE_UNUSED, char **argv)
}
#if !ENABLE_FEATURE_FANCY_ECHO
/* SUSv3 specifies that octal escapes must begin with '0'. */
- if ( (((unsigned char)*arg) - '1') >= 7)
+ if ( ((int)(unsigned char)(*arg) - '0') >= 8) /* '8' or bigger */
#endif
{
/* Since SUSv3 mandates a first digit of 0, 4-digit octals
* of the form \0### are accepted. */
- if (*arg == '0' && ((unsigned char)(arg[1]) - '0') < 8) {
- arg++;
+ if (*arg == '0') {
+ /* NB: don't turn "...\0" into "...\" */
+ if (arg[1] && ((unsigned char)(arg[1]) - '0') < 8) {
+ arg++;
+ }
}
- /* bb_process_escape_sequence can handle nul correctly */
+ /* bb_process_escape_sequence handles NUL correctly
+ * ("...\" case). */
c = bb_process_escape_sequence(&arg);
}
}