From fb630e048c6efabe4d46b1b125fe7348062d38e6 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Mon, 6 Feb 2012 03:13:42 -0500 Subject: [PATCH] tag: die when listing missing or corrupt objects We don't usually bother looking at tagged objects at all when listing. However, if "-n" is specified, we open the objects to read the annotations of the tags. If we fail to read an object, or if the object has zero length, we simply silently return. The first case is an indication of a broken or corrupt repo, and we should notify the user of the error. The second case is OK to silently ignore; however, the existing code leaked the buffer returned by read_sha1_file. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- builtin/tag.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/builtin/tag.c b/builtin/tag.c index 391160cf8..1e27f5c3d 100644 --- a/builtin/tag.c +++ b/builtin/tag.c @@ -93,8 +93,12 @@ static void show_tag_lines(const unsigned char *sha1, int lines) size_t len; buf = read_sha1_file(sha1, &type, &size); - if (!buf || !size) + if (!buf) + die_errno("unable to read object %s", sha1_to_hex(sha1)); + if (!size) { + free(buf); return; + } /* skip header */ sp = strstr(buf, "\n\n"); -- 2.30.2