From: Ævar Arnfjörð Bjarmason Date: Thu, 27 Jan 2022 05:26:45 +0000 (+0100) Subject: object-name: explicitly handle bad tags in show_ambiguous_object() X-Git-Tag: v2.36.0-rc0~115^2~4 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=667a560be7ae977e61d81ca4bcb95ded1cdacc9c;p=thirdparty%2Fgit.git object-name: explicitly handle bad tags in show_ambiguous_object() Follow-up the handling of OBJ_BAD in the preceding commit and explicitly handle those cases where parse_tag() fails, or we don't end up with a non-NULL pointer in in tag->tag. If we run into such a tag we'd previously be silent about it. We really should also be handling these batter in parse_tag_buffer() by being more eager to emit an error(), instead of silently aborting with "return -1;". One example of such a tag is the one that's tested for in "t3800-mktag.sh", where the code takes the "size < the_hash_algo->hexsz + 24" branch. But in lieu of earlier missing "error" output let's show the user something to indicate why we're not showing a tag message in these cases, now instead of showing: hint: deadbeef tag We'll instead display: hint: deadbeef tag [tag could not be parsed] Signed-off-by: Ævar Arnfjörð Bjarmason Signed-off-by: Junio C Hamano --- diff --git a/object-name.c b/object-name.c index 9750634ee7..298b742bac 100644 --- a/object-name.c +++ b/object-name.c @@ -382,6 +382,8 @@ static int show_ambiguous_object(const struct object_id *oid, void *data) struct tag *tag = lookup_tag(ds->repo, oid); if (!parse_tag(tag) && tag->tag) strbuf_addf(&desc, " %s", tag->tag); + else + strbuf_addstr(&desc, " [tag could not be parsed]"); } out: