diff options
author | Jean-Jacques Lafay <jeanjacques.lafay@gmail.com> | 2014-04-24 14:24:39 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-04-25 09:35:20 -0700 |
commit | cbc60b67201e083a4970c8731c5382a575357e36 (patch) | |
tree | 3272cd96773c56c647a03c3bf0f46f784eda4a32 /builtin/merge-base.c | |
parent | 779792a5f24bb4e8049c4f88ad752e70d4a8a080 (diff) | |
download | git-cbc60b67201e083a4970c8731c5382a575357e36.tar.gz git-cbc60b67201e083a4970c8731c5382a575357e36.tar.xz |
git tag --contains: avoid stack overflow
In large repos, the recursion implementation of contains(commit,
commit_list) may result in a stack overflow. Replace the recursion with
a loop to fix it.
This problem is more apparent on Windows than on Linux, where the stack
is more limited by default.
See also this thread on the msysGit list:
https://groups.google.com/d/topic/msysgit/FqT6boJrb2g/discussion
[jes: re-written to imitate the original recursion more closely]
Thomas Braun pointed out several documentation shortcomings.
Tests are run only if ulimit -s is available. This means they cannot
be run on Windows.
Signed-off-by: Jean-Jacques Lafay <jeanjacques.lafay@gmail.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Tested-by: Stepan Kasal <kasal@ucw.cz>
Reviewed-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/merge-base.c')
0 files changed, 0 insertions, 0 deletions