diff options
author | Denys Vlasenko | 2021-07-11 18:16:10 +0200 |
---|---|---|
committer | Denys Vlasenko | 2021-07-11 18:16:10 +0200 |
commit | caa93ecdd3a9b998a69dcbfafdddbc9c58887ec3 (patch) | |
tree | dca825b9488bd52398688ee52b41bc6473fff3e7 | |
parent | f5f336e787fd8367d24de5fe3b341ebffb74c2c2 (diff) | |
download | busybox-caa93ecdd3a9b998a69dcbfafdddbc9c58887ec3.zip busybox-caa93ecdd3a9b998a69dcbfafdddbc9c58887ec3.tar.gz |
awk: fix corner case in awk_printf
Example where it wasn't working:
awk 'BEGIN { printf "qwe %s rty %c uio\n", "a", 0, "c" }'
- the NUL printing in %c caused premature stop of printing.
function old new delta
awk_printf 593 596 +3
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | editors/awk.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/editors/awk.c b/editors/awk.c index 6c60a06..465033f 100644 --- a/editors/awk.c +++ b/editors/awk.c @@ -2359,11 +2359,11 @@ static char *awk_printf(node *n, size_t *len) * printf "%99999s", "BOOM" */ if (c == 'c') { - c = is_numeric(arg) ? getvar_i(arg) : *getvar_s(arg); - s = xasprintf(s, c); - /* + 1 if c == NUL: handle printf "%c" 0 case + char cc = is_numeric(arg) ? getvar_i(arg) : *getvar_s(arg); + s = xasprintf(s, cc); + /* + 1 if cc == NUL: handle printf "%c" 0 case * (and printf "%22c" 0 etc, but still fails for e.g. printf "%-22c" 0) */ - slen = strlen(s) + (c == '\0'); + slen = strlen(s) + (cc == '\0'); } else { if (c == 's') { s = xasprintf(s, getvar_s(arg)); |