]> git.ipfire.org Git - thirdparty/git.git/commit
t8002: fix ambiguous printf conversion specifications
authorJan Palus <jpalus@fastmail.com>
Mon, 20 Jan 2025 11:41:06 +0000 (12:41 +0100)
committerJunio C Hamano <gitster@pobox.com>
Tue, 21 Jan 2025 22:04:26 +0000 (14:04 -0800)
commit2d0ff147e5f6a46554605d137993ba385698eb4e
treeab169e167ff5623d8135f2c45211baf06292934a
parente7fb2ca94556e6aadfc3038afaa1c8cc3525258c
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>
t/t8002-blame.sh