diff options
| author | Edward Thomson <ethomson@edwardthomson.com> | 2022-06-13 22:14:37 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-06-13 22:14:37 -0400 |
| commit | f98dd5438f8d7bfd557b612fdf1605b1c3fb8eaf (patch) | |
| tree | 49bf90d07df60ac5c9ebdede5547f4a733d013cd | |
| parent | 05f211543bda96ba86bef14e8e6521069fb77797 (diff) | |
| parent | cdff2f0237f663e0f68155655a8b66d05c1ec716 (diff) | |
| download | libgit2-f98dd5438f8d7bfd557b612fdf1605b1c3fb8eaf.tar.gz | |
Merge pull request #6321 from libgit2/ethomson/ownership
repo: allow administrator to own the configuration
| -rw-r--r-- | src/libgit2/repository.c | 2 | ||||
| -rw-r--r-- | tests/libgit2/repo/open.c | 5 |
2 files changed, 4 insertions, 3 deletions
diff --git a/src/libgit2/repository.c b/src/libgit2/repository.c index 48a0b70f5..d2484318f 100644 --- a/src/libgit2/repository.c +++ b/src/libgit2/repository.c @@ -512,7 +512,7 @@ static int validate_ownership(const char *repo_path) bool is_safe; int error; - if ((error = git_fs_path_owner_is_current_user(&is_safe, repo_path)) < 0) { + if ((error = git_fs_path_owner_is_system_or_current_user(&is_safe, repo_path)) < 0) { if (error == GIT_ENOTFOUND) error = 0; diff --git a/tests/libgit2/repo/open.c b/tests/libgit2/repo/open.c index 4b6609a81..5c66eca4b 100644 --- a/tests/libgit2/repo/open.c +++ b/tests/libgit2/repo/open.c @@ -484,9 +484,10 @@ void test_repo_open__validates_dir_ownership(void) cl_git_pass(git_repository_open(&repo, "empty_standard_repo")); git_repository_free(repo); - /* When the system user owns the repo config, fail */ + /* When the system user owns the repo config, also acceptable */ git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_SYSTEM); - cl_git_fail(git_repository_open(&repo, "empty_standard_repo")); + cl_git_pass(git_repository_open(&repo, "empty_standard_repo")); + git_repository_free(repo); /* When an unknown user owns the repo config, fail */ git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_OTHER); |