Merge pull request #774 from benstraub/revparse-colon-segfault

Fix potential segfault in revparse.
This commit is contained in:
Vicent Martí 2012-06-19 09:27:40 -07:00
commit 1b0ef5aa03
2 changed files with 7 additions and 1 deletions

View File

@ -406,7 +406,7 @@ static int handle_caret_syntax(git_object **out, git_repository *repo, git_objec
*out = newobj2;
return 0;
}
/* {/...} -> Walk all commits until we see a commit msg that matches the phrase. */
if (movement[1] == '/') {
int retcode = GIT_ERROR;
@ -550,6 +550,11 @@ static int oid_for_tree_path(git_oid *out, git_tree *tree, git_repository *repo,
}
}
if (!entry) {
giterr_set(GITERR_INVALID, "Invalid tree path '%s'", path);
return GIT_ERROR;
}
git_oid_cpy(out, git_tree_entry_id(entry));
git__free(alloc);
return 0;

View File

@ -163,6 +163,7 @@ void test_refs_revparse__colon(void)
cl_git_fail(git_revparse_single(&g_obj, g_repo, ":/"));
cl_git_fail(git_revparse_single(&g_obj, g_repo, ":/not found in any commit"));
cl_git_fail(git_revparse_single(&g_obj, g_repo, ":2:README"));
cl_git_fail(git_revparse_single(&g_obj, g_repo, "master:"));
test_object("subtrees:ab/4.txt", "d6c93164c249c8000205dd4ec5cbca1b516d487f");
test_object("subtrees:ab/de/fgh/1.txt", "1f67fc4386b2d171e0d21be1c447e12660561f9b");