diff options
author | Jan Palus <jpalus@fastmail.com> | 2025-01-20 12:41:06 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2025-01-21 14:04:26 -0800 |
commit | 2d0ff147e5f6a46554605d137993ba385698eb4e (patch) | |
tree | ab169e167ff5623d8135f2c45211baf06292934a /builtin/commit.c | |
parent | e7fb2ca94556e6aadfc3038afaa1c8cc3525258c (diff) |
t8002: fix ambiguous printf conversion specifications
In e7fb2ca945 (builtin/blame: fix out-of-bounds write with blank
boundary commits, 2025-01-10), we have introduced two new tests that
expect a certain amount of padding. This padding is generated via
printf using the "%0.s" conversion specification. That directive is
ambiguous because it might be interpreted as field width (most shells)
or 0-padding flag for numeric fields (coreutils).
Fix this issue by using "%${N}s" instead, which is already being
used in other tests (i.e. t5300, t0450) and is unambiguous.
Helped-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Jan Palus <jpalus@fastmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions