X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=tag.c;h=22deb243ad58b2c57fb8652fe2d08c571ee3e781;hb=2a45925180152b3e54248de4d7f69e8cc5c4051f;hp=3a71dd416ad18d86ed28ed6593e4588151cc43ea;hpb=d0d0d0bd3c2c4591ffbc292d7e082e8ad8f2057f;p=git.git diff --git a/tag.c b/tag.c index 3a71dd41..22deb243 100644 --- a/tag.c +++ b/tag.c @@ -21,11 +21,8 @@ struct tag *lookup_tag(unsigned char *sha1) return (struct tag *) obj; } -int parse_tag(struct tag *item) +int parse_tag_buffer(struct tag *item, void *data, unsigned long size) { - char type[20]; - void *data, *bufptr; - unsigned long size; int typelen, taglen; unsigned char object[20]; const char *type_line, *tag_line, *sig_line; @@ -33,20 +30,11 @@ int parse_tag(struct tag *item) if (item->object.parsed) return 0; item->object.parsed = 1; - data = bufptr = read_sha1_file(item->object.sha1, type, &size); - if (!data) - return error("Could not read %s", - sha1_to_hex(item->object.sha1)); - if (strcmp(type, tag_type)) { - free(data); - return error("Object %s not a tag", - sha1_to_hex(item->object.sha1)); - } if (size < 64) - goto err; + return -1; if (memcmp("object ", data, 7) || get_sha1_hex(data + 7, object)) - goto err; + return -1; item->tagged = parse_object(object); if (item->tagged) @@ -54,29 +42,47 @@ int parse_tag(struct tag *item) type_line = data + 48; if (memcmp("\ntype ", type_line-1, 6)) - goto err; + return -1; tag_line = strchr(type_line, '\n'); if (!tag_line || memcmp("tag ", ++tag_line, 4)) - goto err; + return -1; sig_line = strchr(tag_line, '\n'); if (!sig_line) - goto err; + return -1; sig_line++; typelen = tag_line - type_line - strlen("type \n"); if (typelen >= 20) - goto err; + return -1; taglen = sig_line - tag_line - strlen("tag \n"); item->tag = xmalloc(taglen + 1); memcpy(item->tag, tag_line + 4, taglen); item->tag[taglen] = '\0'; - free(data); return 0; +} -err: +int parse_tag(struct tag *item) +{ + char type[20]; + void *data; + unsigned long size; + int ret; + + if (item->object.parsed) + return 0; + data = read_sha1_file(item->object.sha1, type, &size); + if (!data) + return error("Could not read %s", + sha1_to_hex(item->object.sha1)); + if (strcmp(type, tag_type)) { + free(data); + return error("Object %s not a tag", + sha1_to_hex(item->object.sha1)); + } + ret = parse_tag_buffer(item, data, size); free(data); - return -1; + return ret; }