Commit f51f6646 by Edward Thomson

Revert "repo: allow administrator to own the configuration"

This reverts commit cdff2f02.

This change erroneously allowed system users to own a worktree; this
should only be allowed when the current user is in the Administrator
group on Windows as well.
parent 92ffdd2c
...@@ -512,7 +512,7 @@ static int validate_ownership(const char *repo_path) ...@@ -512,7 +512,7 @@ static int validate_ownership(const char *repo_path)
bool is_safe; bool is_safe;
int error; int error;
if ((error = git_fs_path_owner_is_system_or_current_user(&is_safe, repo_path)) < 0) { if ((error = git_fs_path_owner_is_current_user(&is_safe, repo_path)) < 0) {
if (error == GIT_ENOTFOUND) if (error == GIT_ENOTFOUND)
error = 0; error = 0;
......
...@@ -484,10 +484,9 @@ void test_repo_open__validates_dir_ownership(void) ...@@ -484,10 +484,9 @@ void test_repo_open__validates_dir_ownership(void)
cl_git_pass(git_repository_open(&repo, "empty_standard_repo")); cl_git_pass(git_repository_open(&repo, "empty_standard_repo"));
git_repository_free(repo); git_repository_free(repo);
/* When the system user owns the repo config, also acceptable */ /* When the system user owns the repo config, fail */
git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_SYSTEM); git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_SYSTEM);
cl_git_pass(git_repository_open(&repo, "empty_standard_repo")); cl_git_fail(git_repository_open(&repo, "empty_standard_repo"));
git_repository_free(repo);
/* When an unknown user owns the repo config, fail */ /* When an unknown user owns the repo config, fail */
git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_OTHER); git_fs_path__set_owner(GIT_FS_PATH_MOCK_OWNER_OTHER);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment