summaryrefslogtreecommitdiff
path: root/builtin/commit.c
diff options
context:
space:
mode:
authorJan Palus <jpalus@fastmail.com>2025-01-20 12:41:06 +0100
committerJunio C Hamano <gitster@pobox.com>2025-01-21 14:04:26 -0800
commit2d0ff147e5f6a46554605d137993ba385698eb4e (patch)
treeab169e167ff5623d8135f2c45211baf06292934a /builtin/commit.c
parente7fb2ca94556e6aadfc3038afaa1c8cc3525258c (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