From d52dfc25d592b3e88ace4bcd30aff1fb665b88e5 Mon Sep 17 00:00:00 2001 From: Florian Zschocke Date: Mon, 10 Jun 2019 16:37:15 +0200 Subject: Do not resolve symbolic links to repositories. When symbolic links under the base repository folder point to repositories outside the base repository folder, the forming of relative repository names failed and resulted in NullPointerExceptions. Create the relative path by not following symbolic links, i.e. the link name is taken as is and not resolved to the external path. This also changes the whole `exactPath` method to work on Paths, instead of Files. Fixes #891 and fixes #837. --- src/test/java/com/gitblit/tests/JGitUtilsTest.java | 117 +++++++++++++++++++++ 1 file changed, 117 insertions(+) (limited to 'src/test/java/com/gitblit/tests') diff --git a/src/test/java/com/gitblit/tests/JGitUtilsTest.java b/src/test/java/com/gitblit/tests/JGitUtilsTest.java index b133a8c8..5f083555 100644 --- a/src/test/java/com/gitblit/tests/JGitUtilsTest.java +++ b/src/test/java/com/gitblit/tests/JGitUtilsTest.java @@ -17,6 +17,10 @@ package com.gitblit.tests; import java.io.File; import java.io.FileOutputStream; +import java.io.IOException; +import java.nio.file.Files; +import java.nio.file.Path; +import java.nio.file.Paths; import java.text.SimpleDateFormat; import java.util.Arrays; import java.util.Date; @@ -51,6 +55,8 @@ import com.gitblit.utils.JGitUtils; import com.gitblit.utils.JnaUtils; import com.gitblit.utils.StringUtils; +import static org.junit.Assume.assumeTrue; + public class JGitUtilsTest extends GitblitUnitTest { @Test @@ -74,6 +80,117 @@ public class JGitUtilsTest extends GitblitUnitTest { assertTrue("No repositories found in " + GitBlitSuite.REPOSITORIES, list.size() > 0); } + @Test + public void testFindRepositoriesSymLinked() { + + String reposdirName = "gitrepos"; + String repositoryName = "test-linked.git"; + File extrepodir = new File(GitBlitSuite.REPOSITORIES.getParent(), reposdirName); + Path symlink = null; + Path alink = null; + try { + Path link = Paths.get( GitBlitSuite.REPOSITORIES.toString(), "test-rln.git"); + Path target = Paths.get("../" + reposdirName,repositoryName); + symlink = Files.createSymbolicLink(link, target); + + link = Paths.get( GitBlitSuite.REPOSITORIES.toString(), "test-ln.git"); + target = Paths.get(extrepodir.getCanonicalPath(), repositoryName); + alink = Files.createSymbolicLink(link, target); + } + catch (UnsupportedOperationException e) { + assumeTrue("No symbolic links supported.", false); + } + catch (IOException ioe) { + try { + if (symlink != null) Files.delete(symlink); + if (alink != null) Files.delete(alink); + } + catch (IOException ignored) {} + fail(ioe.getMessage()); + } + + + Path extDir = null; + Repository repository = null; + + String testDirName = "test-linked"; + String testTestDirName = "test-linked/test"; + Path testDir = Paths.get(GitBlitSuite.REPOSITORIES.toString(),testDirName); + Path testTestDir = Paths.get(GitBlitSuite.REPOSITORIES.toString(),testTestDirName); + Path linkTestRepo = null; + Path linkTestTestRepo = null; + Path linkExtDir = null; + try { + List list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null); + int preSize = list.size(); + + // Create test repo. This will make the link targets exist, so that the number of repos increases by two. + extDir = Files.createDirectory(extrepodir.toPath()); + repository = JGitUtils.createRepository(extrepodir, repositoryName); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null); + assertEquals("No linked repositories found in " + GitBlitSuite.REPOSITORIES, 2, (list.size() - preSize)); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, Arrays.asList(".*ln\\.git")); + assertEquals("Filtering out linked repos failed.", preSize, list.size()); + + // Create subdirectories and place links into them + Files.createDirectories(testTestDir); + + Path target = Paths.get(extrepodir.getCanonicalPath(), repositoryName); + Path link = Paths.get(testDir.toString(), "test-ln-one.git"); + linkTestRepo = Files.createSymbolicLink(link, target); + link = Paths.get(testTestDir.toString(), "test-ln-two.git"); + linkTestTestRepo = Files.createSymbolicLink(link, target); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null); + assertEquals("No linked repositories found in subdirectories of " + GitBlitSuite.REPOSITORIES, 4, (list.size() - preSize)); + assertTrue("Did not find linked repo test-ln-one.git", list.contains(testDirName + "/test-ln-one.git")); + assertTrue("Did not find linked repo test-ln-two.git", list.contains(testTestDirName + "/test-ln-two.git")); + list = JGitUtils.getRepositoryList(new File(testDir.toString()), true, true, -1, null); + assertEquals("No linked repositories found in subdirectories of " + testDir, 2, list.size()); + assertTrue("Did not find linked repo test-ln-one.git", list.contains("test-ln-one.git")); + assertTrue("Did not find linked repo test-ln-two.git", list.contains("test/test-ln-two.git")); + + + // Create link to external directory with repos + target = Paths.get(extrepodir.getCanonicalPath()); + link = Paths.get(testDir.toString(), "test-linked"); + linkExtDir = Files.createSymbolicLink(link, target); + + list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null); + assertEquals("No repositories found in linked subdirectories of " + GitBlitSuite.REPOSITORIES, 5, (list.size() - preSize)); + assertTrue("Did not find repo in linked subfolder.", list.contains(testDirName + "/test-linked/" + repositoryName)); + + list = JGitUtils.getRepositoryList(new File(testDir.toString()), true, true, -1, null); + assertEquals("No repositories found in linked subdirectories of " + testDir, 3, list.size()); + assertTrue("Did not find repo in linked subfolder.", list.contains("test-linked/" + repositoryName)); + + } catch (IOException e) { + fail(e.toString()); + } finally { + try { + if (repository != null) { + repository.close(); + RepositoryCache.close(repository); + FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE); + } + if (extDir != null) Files.delete(extDir); + if (symlink != null) Files.delete(symlink); + if (alink != null) Files.delete(alink); + + if (linkExtDir != null) Files.deleteIfExists(linkExtDir); + if (linkTestTestRepo != null) Files.deleteIfExists(linkTestTestRepo); + if (linkTestRepo != null) Files.deleteIfExists(linkTestRepo); + + Files.deleteIfExists(testTestDir); + Files.deleteIfExists(testDir); + } + catch (IOException ignored) {} + } + + } + @Test public void testFindExclusions() { List list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, null); -- cgit v1.2.3