[PATCH] Fix memory leaks in git-fsck-cache

This patch fixes memory leaks in parse_object() and related functions;
these leaks were very noticeable when running git-fsck-cache.

Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
Sergey Vlasov 2005-05-04 21:44:15 +04:00 committed by Linus Torvalds
parent 705acc5c94
commit 13019d4136
5 changed files with 27 additions and 11 deletions

1
blob.c
View File

@ -34,6 +34,7 @@ int parse_blob(struct blob *item)
if (!buffer) if (!buffer)
return error("Could not read %s", return error("Could not read %s",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
free(buffer);
if (strcmp(type, blob_type)) if (strcmp(type, blob_type))
return error("Object %s not a blob", return error("Object %s not a blob",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));

View File

@ -54,9 +54,11 @@ int parse_commit(struct commit *item)
if (!buffer) if (!buffer)
return error("Could not read %s", return error("Could not read %s",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
if (strcmp(type, commit_type)) if (strcmp(type, commit_type)) {
free(buffer);
return error("Object %s not a commit", return error("Object %s not a commit",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
}
get_sha1_hex(bufptr + 5, parent); get_sha1_hex(bufptr + 5, parent);
item->tree = lookup_tree(parent); item->tree = lookup_tree(parent);
if (item->tree) if (item->tree)

View File

@ -107,11 +107,12 @@ struct object *parse_object(unsigned char *sha1)
char type[100]; char type[100];
unsigned long size; unsigned long size;
void *buffer = unpack_sha1_file(map, mapsize, type, &size); void *buffer = unpack_sha1_file(map, mapsize, type, &size);
munmap(map, mapsize);
if (!buffer) if (!buffer)
return NULL; return NULL;
if (check_sha1_signature(sha1, buffer, size, type) < 0) if (check_sha1_signature(sha1, buffer, size, type) < 0)
printf("sha1 mismatch %s\n", sha1_to_hex(sha1)); printf("sha1 mismatch %s\n", sha1_to_hex(sha1));
munmap(map, mapsize); free(buffer);
if (!strcmp(type, "blob")) { if (!strcmp(type, "blob")) {
struct blob *ret = lookup_blob(sha1); struct blob *ret = lookup_blob(sha1);
parse_blob(ret); parse_blob(ret);

21
tag.c
View File

@ -37,37 +37,44 @@ int parse_tag(struct tag *item)
if (!data) if (!data)
return error("Could not read %s", return error("Could not read %s",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
if (strcmp(type, tag_type)) if (strcmp(type, tag_type)) {
free(data);
return error("Object %s not a tag", return error("Object %s not a tag",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
}
if (size < 64) if (size < 64)
return -1; goto err;
if (memcmp("object ", data, 7) || get_sha1_hex(data + 7, object)) if (memcmp("object ", data, 7) || get_sha1_hex(data + 7, object))
return -1; goto err;
item->tagged = parse_object(object); item->tagged = parse_object(object);
type_line = data + 48; type_line = data + 48;
if (memcmp("\ntype ", type_line-1, 6)) if (memcmp("\ntype ", type_line-1, 6))
return -1; goto err;
tag_line = strchr(type_line, '\n'); tag_line = strchr(type_line, '\n');
if (!tag_line || memcmp("tag ", ++tag_line, 4)) if (!tag_line || memcmp("tag ", ++tag_line, 4))
return -1; goto err;
sig_line = strchr(tag_line, '\n'); sig_line = strchr(tag_line, '\n');
if (!sig_line) if (!sig_line)
return -1; goto err;
sig_line++; sig_line++;
typelen = tag_line - type_line - strlen("type \n"); typelen = tag_line - type_line - strlen("type \n");
if (typelen >= 20) if (typelen >= 20)
return -1; goto err;
taglen = sig_line - tag_line - strlen("tag \n"); taglen = sig_line - tag_line - strlen("tag \n");
item->tag = xmalloc(taglen + 1); item->tag = xmalloc(taglen + 1);
memcpy(item->tag, tag_line + 4, taglen); memcpy(item->tag, tag_line + 4, taglen);
item->tag[taglen] = '\0'; item->tag[taglen] = '\0';
free(data);
return 0; return 0;
err:
free(data);
return -1;
} }

9
tree.c
View File

@ -101,9 +101,11 @@ int parse_tree(struct tree *item)
if (!buffer) if (!buffer)
return error("Could not read %s", return error("Could not read %s",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
if (strcmp(type, tree_type)) if (strcmp(type, tree_type)) {
free(buffer);
return error("Object %s not a tree", return error("Object %s not a tree",
sha1_to_hex(item->object.sha1)); sha1_to_hex(item->object.sha1));
}
list_p = &item->entries; list_p = &item->entries;
while (size) { while (size) {
struct object *obj; struct object *obj;
@ -113,8 +115,10 @@ int parse_tree(struct tree *item)
char *path = strchr(bufptr, ' '); char *path = strchr(bufptr, ' ');
unsigned int mode; unsigned int mode;
if (size < len + 20 || !path || if (size < len + 20 || !path ||
sscanf(bufptr, "%o", &mode) != 1) sscanf(bufptr, "%o", &mode) != 1) {
free(buffer);
return -1; return -1;
}
entry = xmalloc(sizeof(struct tree_entry_list)); entry = xmalloc(sizeof(struct tree_entry_list));
entry->name = strdup(path + 1); entry->name = strdup(path + 1);
@ -138,5 +142,6 @@ int parse_tree(struct tree *item)
*list_p = entry; *list_p = entry;
list_p = &entry->next; list_p = &entry->next;
} }
free(buffer);
return 0; return 0;
} }