mirror of
https://git.proxmox.com/git/libgit2
synced 2026-01-06 18:52:21 +00:00
Parse ref oids without trailing newline
This commit is contained in:
parent
e4607392b5
commit
6ab6829097
@ -173,8 +173,8 @@ static int loose_parse_oid(git_oid *oid, git_buf *file_content)
|
||||
|
||||
buffer = (char *)file_content->ptr;
|
||||
|
||||
/* File format: 40 chars (OID) + newline */
|
||||
if (git_buf_len(file_content) < GIT_OID_HEXSZ + 1)
|
||||
/* File format: 40 chars (OID) */
|
||||
if (git_buf_len(file_content) < GIT_OID_HEXSZ)
|
||||
goto corrupt;
|
||||
|
||||
if (git_oid_fromstr(oid, buffer) < 0)
|
||||
@ -184,7 +184,10 @@ static int loose_parse_oid(git_oid *oid, git_buf *file_content)
|
||||
if (*buffer == '\r')
|
||||
buffer++;
|
||||
|
||||
if (*buffer != '\n')
|
||||
if (*buffer == '\n')
|
||||
buffer++;
|
||||
|
||||
if (*buffer != '\0')
|
||||
goto corrupt;
|
||||
|
||||
return 0;
|
||||
|
||||
@ -107,7 +107,7 @@ void test_network_remotelocal__retrieve_advertised_references(void)
|
||||
|
||||
cl_git_pass(git_remote_ls(remote, &count_ref__cb, &how_many_refs));
|
||||
|
||||
cl_assert_equal_i(how_many_refs, 23);
|
||||
cl_assert_equal_i(how_many_refs, 24);
|
||||
}
|
||||
|
||||
void test_network_remotelocal__retrieve_advertised_references_from_spaced_repository(void)
|
||||
@ -121,7 +121,7 @@ void test_network_remotelocal__retrieve_advertised_references_from_spaced_reposi
|
||||
|
||||
cl_git_pass(git_remote_ls(remote, &count_ref__cb, &how_many_refs));
|
||||
|
||||
cl_assert_equal_i(how_many_refs, 23);
|
||||
cl_assert_equal_i(how_many_refs, 24);
|
||||
|
||||
git_remote_free(remote); /* Disconnect from the "spaced repo" before the cleanup */
|
||||
remote = NULL;
|
||||
|
||||
@ -47,7 +47,7 @@ static void assert_retrieval(unsigned int flags, unsigned int expected_count)
|
||||
|
||||
void test_refs_branches_foreach__retrieve_all_branches(void)
|
||||
{
|
||||
assert_retrieval(GIT_BRANCH_LOCAL | GIT_BRANCH_REMOTE, 11);
|
||||
assert_retrieval(GIT_BRANCH_LOCAL | GIT_BRANCH_REMOTE, 12);
|
||||
}
|
||||
|
||||
void test_refs_branches_foreach__retrieve_remote_branches(void)
|
||||
@ -57,7 +57,7 @@ void test_refs_branches_foreach__retrieve_remote_branches(void)
|
||||
|
||||
void test_refs_branches_foreach__retrieve_local_branches(void)
|
||||
{
|
||||
assert_retrieval(GIT_BRANCH_LOCAL, 9);
|
||||
assert_retrieval(GIT_BRANCH_LOCAL, 10);
|
||||
}
|
||||
|
||||
struct expectations {
|
||||
|
||||
@ -45,8 +45,8 @@ static void assert_retrieval(const char *glob, unsigned int flags, int expected_
|
||||
|
||||
void test_refs_foreachglob__retrieve_all_refs(void)
|
||||
{
|
||||
/* 7 heads (including one packed head) + 1 note + 2 remotes + 6 tags */
|
||||
assert_retrieval("*", GIT_REF_LISTALL, 18);
|
||||
/* 8 heads (including one packed head) + 1 note + 2 remotes + 6 tags */
|
||||
assert_retrieval("*", GIT_REF_LISTALL, 19);
|
||||
}
|
||||
|
||||
void test_refs_foreachglob__retrieve_remote_branches(void)
|
||||
@ -56,7 +56,7 @@ void test_refs_foreachglob__retrieve_remote_branches(void)
|
||||
|
||||
void test_refs_foreachglob__retrieve_local_branches(void)
|
||||
{
|
||||
assert_retrieval("refs/heads/*", GIT_REF_LISTALL, 9);
|
||||
assert_retrieval("refs/heads/*", GIT_REF_LISTALL, 10);
|
||||
}
|
||||
|
||||
void test_refs_foreachglob__retrieve_partially_named_references(void)
|
||||
|
||||
@ -193,6 +193,18 @@ void test_refs_read__loose_first(void)
|
||||
git_reference_free(reference);
|
||||
}
|
||||
|
||||
void test_refs_read__chomped(void)
|
||||
{
|
||||
git_reference *test, *chomped;
|
||||
|
||||
cl_git_pass(git_reference_lookup(&test, g_repo, "refs/heads/test"));
|
||||
cl_git_pass(git_reference_lookup(&chomped, g_repo, "refs/heads/chomped"));
|
||||
cl_git_pass(git_oid_cmp(git_reference_oid(test), git_reference_oid(chomped)));
|
||||
|
||||
git_reference_free(test);
|
||||
git_reference_free(chomped);
|
||||
}
|
||||
|
||||
void test_refs_read__unfound_return_ENOTFOUND(void)
|
||||
{
|
||||
git_reference *reference;
|
||||
|
||||
1
tests-clar/resources/testrepo.git/refs/heads/chomped
Normal file
1
tests-clar/resources/testrepo.git/refs/heads/chomped
Normal file
@ -0,0 +1 @@
|
||||
e90810b8df3e80c413d903f631643c716887138d
|
||||
Loading…
Reference in New Issue
Block a user