diff options
author | Jeff King <peff@peff.net> | 2020-12-01 13:11:38 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-12-02 14:30:39 -0800 |
commit | 91aef030152d121f6b4bc3b933c696073ba073e2 (patch) | |
tree | 576aabc81144240c2a5574c481f7fa77a64d54f3 /t/t3509-cherry-pick-merge-df.sh | |
parent | banned.h: mark non-reentrant gmtime, etc as banned (diff) | |
download | tgif-91aef030152d121f6b4bc3b933c696073ba073e2.tar.xz |
banned.h: mark ctime_r() and asctime_r() as banned
The ctime_r() and asctime_r() functions are reentrant, but have
no check that the buffer we pass in is long enough (the manpage says it
"should have room for at least 26 bytes"). Since this is such an
easy-to-get-wrong interface, and since we have the much safer strftime()
as well as its more convenient strbuf_addftime() wrapper, let's ban both
of those.
Signed-off-by: Jeff King <peff@peff.net>
Reviewed-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t3509-cherry-pick-merge-df.sh')
0 files changed, 0 insertions, 0 deletions