| commit | fb630e048c6efabe4d46b1b125fe7348062d38e6 | |
| author | Jeff King <peff@peff.net> | |
| Mon, 6 Feb 2012 08:13:42 +0000 (6 03:13 -0500) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Mon, 6 Feb 2012 18:00:51 +0000 (6 10:00 -0800) | ||
| tree | 1034df77904ed92fd9d0049f9eeec6de08324c2c | treesnapshot (tar.gz zip) |
| parent | ca51699961664890fdaabd276af539e6b3514053 | commitdiff |
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 <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
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 <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| builtin/tag.c | diffblobblamehistory |