diff options
author | Patrick Steinhardt <ps@pks.im> | 2017-05-19 09:21:17 +0200 |
---|---|---|
committer | Patrick Steinhardt <ps@pks.im> | 2017-05-19 09:44:44 +0200 |
commit | 2696c5c3ece1a28c545d6e59b0d121480e9f977d (patch) | |
tree | 5a9f2a5fe87ce832fb7d9cb9b752c1f82703220f /src/repository.c | |
parent | 9f9fd05f1cb9278a34a265c49c8567b526e48afd (diff) | |
download | libgit2-2696c5c3ece1a28c545d6e59b0d121480e9f977d.tar.gz |
repository: make check if repo is a worktree more strict
To determine if a repository is a worktree or not, we currently check
for the existence of a "gitdir" file inside of the repository's gitdir.
While this is sufficient for non-broken repositories, we have at least
one case of a subtly broken repository where there exists a gitdir file
inside of a gitmodule. This will cause us to misidentify the submodule
as a worktree.
While this is not really a fault of ours, we can do better here by
observing that a repository can only ever be a worktree iff its common
directory and dotgit directory are different. This allows us to make our
check whether a repo is a worktree or not more strict by doing a simple
string comparison of these two directories. This will also allow us to
do the right thing in the above case of a broken repository, as for
submodules these directories will be the same. At the same time, this
allows us to skip the `stat` check for the "gitdir" file for most
repositories.
Diffstat (limited to 'src/repository.c')
-rw-r--r-- | src/repository.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/repository.c b/src/repository.c index fec476396..18c6eed76 100644 --- a/src/repository.c +++ b/src/repository.c @@ -763,6 +763,13 @@ static int repo_is_worktree(unsigned *out, const git_repository *repo) git_buf gitdir_link = GIT_BUF_INIT; int error; + /* Worktrees cannot have the same commondir and gitdir */ + if (repo->commondir && repo->gitdir + && !strcmp(repo->commondir, repo->gitdir)) { + *out = 0; + return 0; + } + if ((error = git_buf_joinpath(&gitdir_link, repo->gitdir, "gitdir")) < 0) return -1; |