Fix set-gid bit clearing under Linux when effective gid is different from f...

Florian Zschocke

2013-08-21 b724448b589d60a9a7dda60cf30741048c98e199

 src/test/java/com/gitblit/tests/JGitUtilsTest.java

@@ -52,6 +52,7 @@
import com.gitblit.models.RefModel;

import com.gitblit.utils.CompressionUtils;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.JnaUtils;

import com.gitblit.utils.StringUtils;



public class JGitUtilsTest {

@@ -118,11 +119,11 @@


   @Test

   public void testLastCommit() throws Exception {

      assertEquals(new Date(0), JGitUtils.getLastChange(null));

      assertEquals(new Date(0), JGitUtils.getLastChange(null).when);



      Repository repository = GitBlitSuite.getHelloworldRepository();

      assertTrue(JGitUtils.getCommit(repository, null) != null);

      Date date = JGitUtils.getLastChange(repository);

      Date date = JGitUtils.getLastChange(repository).when;

      repository.close();

      assertNotNull("Could not get last repository change date!", date);

   }

@@ -140,7 +141,7 @@
         assertNull(JGitUtils.getFirstCommit(repository, null));

         assertEquals(folder.lastModified(), JGitUtils.getFirstChange(repository, null)

               .getTime());

         assertEquals(folder.lastModified(), JGitUtils.getLastChange(repository).getTime());

         assertEquals(folder.lastModified(), JGitUtils.getLastChange(repository).when.getTime());

         assertNull(JGitUtils.getCommit(repository, null));

         repository.close();

         RepositoryCache.close(repository);

@@ -149,6 +150,139 @@
   }



   @Test

   public void testCreateRepositoryShared() throws Exception {

      String[] repositories = { "NewSharedTestRepository.git" };

      for (String repositoryName : repositories) {

         Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES,

               repositoryName, "group");

         File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName),

               FS.DETECTED);

         assertNotNull(repository);

         assertFalse(JGitUtils.hasCommits(repository));

         assertNull(JGitUtils.getFirstCommit(repository, null));



         assertEquals("1", repository.getConfig().getString("core", null, "sharedRepository"));



         assertTrue(folder.exists());

         if (! JnaUtils.isWindows()) {

            int mode = JnaUtils.getFilemode(folder);

            assertEquals(JnaUtils.S_ISGID, mode & JnaUtils.S_ISGID);

            assertEquals(JnaUtils.S_IRWXG, mode & JnaUtils.S_IRWXG);



            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/HEAD");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);



            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/config");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);

         }



         repository.close();

         RepositoryCache.close(repository);

         FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE);

      }

   }



   @Test

   public void testCreateRepositorySharedCustom() throws Exception {

      String[] repositories = { "NewSharedTestRepository.git" };

      for (String repositoryName : repositories) {

         Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES,

               repositoryName, "660");

         File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName),

               FS.DETECTED);

         assertNotNull(repository);

         assertFalse(JGitUtils.hasCommits(repository));

         assertNull(JGitUtils.getFirstCommit(repository, null));



         assertEquals("0660", repository.getConfig().getString("core", null, "sharedRepository"));



         assertTrue(folder.exists());

         if (! JnaUtils.isWindows()) {

            int mode = JnaUtils.getFilemode(folder);

            assertEquals(JnaUtils.S_ISGID, mode & JnaUtils.S_ISGID);

            assertEquals(JnaUtils.S_IRWXG, mode & JnaUtils.S_IRWXG);

            assertEquals(0, mode & JnaUtils.S_IRWXO);



            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/HEAD");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);

            assertEquals(0, mode & JnaUtils.S_IRWXO);



            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/config");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);

            assertEquals(0, mode & JnaUtils.S_IRWXO);

         }



         repository.close();

         RepositoryCache.close(repository);

         FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE);

      }

   }



   @Test

   public void testCreateRepositorySharedSgidParent() throws Exception {

      if (! JnaUtils.isWindows()) {

         String repositoryAll = "NewTestRepositoryAll.git";

         String repositoryUmask = "NewTestRepositoryUmask.git";

         String sgidParent = "sgid";

			
         File parent = new File(GitBlitSuite.REPOSITORIES, sgidParent);

         File folder = null;

         boolean parentExisted = parent.exists();

         try {

            if (!parentExisted) {

               assertTrue("Could not create SGID parent folder.", parent.mkdir());

            }

            int mode = JnaUtils.getFilemode(parent);

            assertTrue(mode > 0);

            assertEquals(0, JnaUtils.setFilemode(parent, mode | JnaUtils.S_ISGID | JnaUtils.S_IWGRP));



            Repository repository = JGitUtils.createRepository(parent, repositoryAll, "all");

            folder = FileKey.resolve(new File(parent, repositoryAll), FS.DETECTED);

            assertNotNull(repository);

		
            assertEquals("2", repository.getConfig().getString("core", null, "sharedRepository"));

		
            assertTrue(folder.exists());

            mode = JnaUtils.getFilemode(folder);

            assertEquals(JnaUtils.S_ISGID, mode & JnaUtils.S_ISGID);

	
            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/HEAD");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);

            assertEquals(JnaUtils.S_IROTH, mode & JnaUtils.S_IRWXO);

	
            mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/config");

            assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, mode & JnaUtils.S_IRWXG);

            assertEquals(JnaUtils.S_IROTH, mode & JnaUtils.S_IRWXO);

	
            repository.close();

            RepositoryCache.close(repository);







            repository = JGitUtils.createRepository(parent, repositoryUmask, "umask");

            folder = FileKey.resolve(new File(parent, repositoryUmask), FS.DETECTED);

            assertNotNull(repository);

		
            assertEquals(null, repository.getConfig().getString("core", null, "sharedRepository"));

		
            assertTrue(folder.exists());

            mode = JnaUtils.getFilemode(folder);

            assertEquals(JnaUtils.S_ISGID, mode & JnaUtils.S_ISGID);

	
            repository.close();

            RepositoryCache.close(repository);

         }

         finally {

            FileUtils.delete(new File(parent, repositoryAll), FileUtils.RECURSIVE | FileUtils.IGNORE_ERRORS);

            FileUtils.delete(new File(parent, repositoryUmask), FileUtils.RECURSIVE | FileUtils.IGNORE_ERRORS);

            if (!parentExisted) {

               FileUtils.delete(parent, FileUtils.RECURSIVE | FileUtils.IGNORE_ERRORS);

            }

         }

      }

   }



   @Test

   public void testRefs() throws Exception {

      Repository repository = GitBlitSuite.getJGitRepository();

      Map<ObjectId, List<RefModel>> map = JGitUtils.getAllRefs(repository);