On 11/1/2014 6:14 PM, John Keeping wrote:
printf(1) supports octal escape sequences in its format argument which
are specified as (from POSIX):
"\ddd", where ddd is a one, two, or three-digit octal number
But the argument to the "%b" format specifier allows:
"\0ddd", where ddd is a zero, one, two, or three-digit octal
number
which is similar to the wording for echo(1) (for XSI-conformant
systems):
\0num Write an 8-bit value that is the zero, one, two, or
three-digit octal number num.
Because conv_escape() handles the first case, applying the second
behaviour in conv_escape_str() must also catch the characters '1'-'7' so
that they are not converted as octal numbers.
Your patch seems to have addressed the clear bugs of the patch in that
other thread. Let me attempt to summarise the status:
- POSIX does not specify the behaviour of \1 in echo and in printf %b.
POSIX does not define the behaviour of escape sequences other than the
ones it explicitly specifies. It does not require \1 to be handled as
\\1. It allows it, but it allows the current dash behaviour too.
To quote from the echo specification: "if any of the operands contain a
backslash ( '\' ) character, the results are implementation-defined",
and the bit about XSI doesn't include an exception for \1.
To quote from the printf %b specification: "The interpretation of a
backslash followed by any other sequence of characters is unspecified."
- bash treats \1 as \\1 in echo, but as \01 in printf %b.
- dash treats \1 as \01 in both echo and in printf %b.
- Your patch makes dash treat \1 as \01 in both echo and printf %b.
- The aim of the patch in the other thread was to make dash be more like
bash.
If that is your aim too, if you want dash to behave like bash, in order
to achieve that the code must no longer be shared between echo and
printf %b. Here is a simple test you can run, where dash is without your
patch, and ./src/dash is with your patch:
$ bash -c 'printf "%b" "\1"' | cat -v
^A
$ dash -c 'printf "%b" "\1"' | cat -v
^A
$ ./src/dash -c 'printf "%b" "\1"' | cat -v
\1
If that isn't your aim, if your aim is only to make dash meet POSIX
requirements, then don't worry, it already does so.
Cheers,
Harald van Dijk
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
