From 61adfd30974a6c49b3d07275e9f2c9fc44bf779c Mon Sep 17 00:00:00 2001 From: Jeff King Date: Fri, 3 Jun 2011 01:11:13 -0400 Subject: [PATCH] consider only branches in guess_remote_head The guess_remote_head function tries to figure out where a remote's HEAD is pointing by comparing the sha1 of the remote's HEAD with the sha1 of various refs found on the remote. However, we were too liberal in matching refs, and would match tags or remote tracking branches, even though these things could not possibly be referenced by the HEAD symbolic ref (since git will detach when checking them out). As a result, a clone of a remote repository with a detached HEAD might write "refs/tags/*" into our local HEAD, which is bogus. The resulting HEAD should be detached. The other related code path is remote.c's get_head_names() (which is used for, among other things, "set-head -a"). This was not affected, however, as that function feeds only refs from refs/heads to guess_remote_head. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- remote.c | 4 +++- t/t5707-clone-detached.sh | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/remote.c b/remote.c index ca42a126a..f073b1ecf 100644 --- a/remote.c +++ b/remote.c @@ -1667,7 +1667,9 @@ struct ref *guess_remote_head(const struct ref *head, /* Look for another ref that points there */ for (r = refs; r; r = r->next) { - if (r != head && !hashcmp(r->old_sha1, head->old_sha1)) { + if (r != head && + !prefixcmp(r->name, "refs/heads/") && + !hashcmp(r->old_sha1, head->old_sha1)) { *tail = copy_ref(r); tail = &((*tail)->next); if (!all) diff --git a/t/t5707-clone-detached.sh b/t/t5707-clone-detached.sh index 6cecd4cd3..d63b1e390 100755 --- a/t/t5707-clone-detached.sh +++ b/t/t5707-clone-detached.sh @@ -41,7 +41,7 @@ test_expect_success 'cloned HEAD matches' ' git --git-dir=detached-tag/.git log -1 --format=%s >actual && test_cmp expect actual ' -test_expect_failure 'cloned HEAD is detached' ' +test_expect_success 'cloned HEAD is detached' ' head_is_detached detached-tag ' -- 2.30.2