summaryrefslogtreecommitdiff
path: root/builtin/ls-tree.c
diff options
context:
space:
mode:
authorJon Seymour <jon.seymour@gmail.com>2011-07-24 14:59:14 (GMT)
committerJunio C Hamano <gitster@pobox.com>2011-07-25 17:50:11 (GMT)
commit04f89259a67ba1ec291f023b70278d41ed665a13 (patch)
treececd5e08d1a365c5940006832a7c5ef836bf291e /builtin/ls-tree.c
parent6c1c14480ddf912fca5e09a6a7e66dee7793d2ea (diff)
downloadgit-04f89259a67ba1ec291f023b70278d41ed665a13.zip
git-04f89259a67ba1ec291f023b70278d41ed665a13.tar.gz
git-04f89259a67ba1ec291f023b70278d41ed665a13.tar.bz2
Ensure git ls-tree exits with a non-zero exit code if read_tree_recursive fails.
In the case of a corrupt repository, git ls-tree may report an error but presently it exits with a code of 0. This change uses the return code of read_tree_recursive instead. Improved-by: Jens Lehmann <Jens.Lehmann@web.de> Signed-off-by: Jon Seymour <jon.seymour@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/ls-tree.c')
-rw-r--r--builtin/ls-tree.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/builtin/ls-tree.c b/builtin/ls-tree.c
index f08c5b0..6b666e1 100644
--- a/builtin/ls-tree.c
+++ b/builtin/ls-tree.c
@@ -173,7 +173,5 @@ int cmd_ls_tree(int argc, const char **argv, const char *prefix)
tree = parse_tree_indirect(sha1);
if (!tree)
die("not a tree object");
- read_tree_recursive(tree, "", 0, 0, &pathspec, show_tree, NULL);
-
- return 0;
+ return !!read_tree_recursive(tree, "", 0, 0, &pathspec, show_tree, NULL);
}