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/JnaUtilsTest.java

@@ -20,6 +20,7 @@
import java.io.File;
import java.io.IOException;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;

import org.apache.commons.io.FileUtils;
@@ -36,65 +37,116 @@
public class JnaUtilsTest {

   @Test
   public void testGetgid() {
      if (JnaUtils.isWindows()) {
         try {
            JnaUtils.getFilemode(GitBlitSuite.REPOSITORIES);
         } catch(UnsupportedOperationException e) {}
      }
      else {
         int gid = JnaUtils.getgid();
         assertTrue(gid >= 0);
         int egid = JnaUtils.getegid();
         assertTrue(egid >= 0);
         assertTrue("Really? You're running unit tests as root?!", gid > 0);
         System.out.println("gid: " + gid + "  egid: " + egid);
      }
   }


   @Test
   public void testGetFilemode() throws IOException {
      String repositoryName = "NewJnaTestRepository.git";
      Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, repositoryName);
      File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), FS.DETECTED);
      assertTrue(folder.exists());
      if (JnaUtils.isWindows()) {
         try {
            JnaUtils.getFilemode(GitBlitSuite.REPOSITORIES);
         } catch(UnsupportedOperationException e) {}
      }
      else {
         String repositoryName = "NewJnaTestRepository.git";
         Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, repositoryName);
         File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), FS.DETECTED);
         assertTrue(folder.exists());

      int mode = JnaUtils.getFilemode(folder);
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_IFDIR, (mode & JnaUtils.S_IFMT)); // directory
      assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR | JnaUtils.S_IXUSR, (mode & JnaUtils.S_IRWXU)); // owner full access
         int mode = JnaUtils.getFilemode(folder);
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_IFDIR, (mode & JnaUtils.S_IFMT)); // directory
         assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR | JnaUtils.S_IXUSR, (mode & JnaUtils.S_IRWXU)); // owner full access

      mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/config");
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_IFREG, (mode & JnaUtils.S_IFMT)); // directory
      assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR, (mode & JnaUtils.S_IRWXU)); // owner full access
         mode = JnaUtils.getFilemode(folder.getAbsolutePath() + "/config");
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_IFREG, (mode & JnaUtils.S_IFMT)); // directory
         assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR, (mode & JnaUtils.S_IRWXU)); // owner full access

      repository.close();
      RepositoryCache.close(repository);
      FileUtils.deleteDirectory(repository.getDirectory());
         repository.close();
         RepositoryCache.close(repository);
         FileUtils.deleteDirectory(repository.getDirectory());
         }
   }


   @Test
   public void testSetFilemode() throws IOException {
      String repositoryName = "NewJnaTestRepository.git";
      Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, repositoryName);
      File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), FS.DETECTED);
      assertTrue(folder.exists());
      if (JnaUtils.isWindows()) {
         try {
            JnaUtils.getFilemode(GitBlitSuite.REPOSITORIES);
         } catch(UnsupportedOperationException e) {}
      }
      else {
         String repositoryName = "NewJnaTestRepository.git";
         Repository repository = JGitUtils.createRepository(GitBlitSuite.REPOSITORIES, repositoryName);
         File folder = FileKey.resolve(new File(GitBlitSuite.REPOSITORIES, repositoryName), FS.DETECTED);
         assertTrue(folder.exists());

      File path = new File(folder, "refs");
      int mode = JnaUtils.getFilemode(path);
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_IFDIR, (mode & JnaUtils.S_IFMT)); // directory
      assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR | JnaUtils.S_IXUSR, (mode & JnaUtils.S_IRWXU)); // owner full access
         File path = new File(folder, "refs");
         int mode = JnaUtils.getFilemode(path);
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_IFDIR, (mode & JnaUtils.S_IFMT)); // directory
         assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR | JnaUtils.S_IXUSR, (mode & JnaUtils.S_IRWXU)); // owner full access

      mode |= JnaUtils.S_ISGID;
      mode |= JnaUtils.S_IRWXG;
      int ret = JnaUtils.setFilemode(path, mode);
      assertEquals(0, ret);
      mode = JnaUtils.getFilemode(path);
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_ISGID, (mode & JnaUtils.S_ISGID)); // set-gid-bit set
      assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP | JnaUtils.S_IXGRP, (mode & JnaUtils.S_IRWXG)); // group full access
         mode |= JnaUtils.S_ISGID;
         mode |= JnaUtils.S_IRWXG;
         int ret = JnaUtils.setFilemode(path, mode);
         assertEquals(0, ret);
         mode = JnaUtils.getFilemode(path);
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_ISGID, (mode & JnaUtils.S_ISGID)); // set-gid-bit set
         assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP | JnaUtils.S_IXGRP, (mode & JnaUtils.S_IRWXG)); // group full access

      path = new File(folder, "config");
      mode = JnaUtils.getFilemode(path.getAbsolutePath());
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_IFREG, (mode & JnaUtils.S_IFMT)); // directory
      assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR, (mode & JnaUtils.S_IRWXU)); // owner full access
         path = new File(folder, "config");
         mode = JnaUtils.getFilemode(path.getAbsolutePath());
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_IFREG, (mode & JnaUtils.S_IFMT)); // directory
         assertEquals(JnaUtils.S_IRUSR | JnaUtils.S_IWUSR, (mode & JnaUtils.S_IRWXU)); // owner full access

      mode |= (JnaUtils.S_IRGRP | JnaUtils.S_IWGRP);
      ret = JnaUtils.setFilemode(path.getAbsolutePath(), mode);
      assertEquals(0, ret);
      mode = JnaUtils.getFilemode(path.getAbsolutePath());
      assertTrue(mode > 0);
      assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, (mode & JnaUtils.S_IRWXG)); // group full access
         mode |= (JnaUtils.S_IRGRP | JnaUtils.S_IWGRP);
         ret = JnaUtils.setFilemode(path.getAbsolutePath(), mode);
         assertEquals(0, ret);
         mode = JnaUtils.getFilemode(path.getAbsolutePath());
         assertTrue(mode > 0);
         assertEquals(JnaUtils.S_IRGRP | JnaUtils.S_IWGRP, (mode & JnaUtils.S_IRWXG)); // group full access

      repository.close();
      RepositoryCache.close(repository);
      FileUtils.deleteDirectory(repository.getDirectory());
         repository.close();
         RepositoryCache.close(repository);
         FileUtils.deleteDirectory(repository.getDirectory());
      }
   }


   @Test
   public void testGetFilestat() {
      if (JnaUtils.isWindows()) {
         try {
            JnaUtils.getFilemode(GitBlitSuite.REPOSITORIES);
         } catch(UnsupportedOperationException e) {}
      }
      else {
         JnaUtils.Filestat stat = JnaUtils.getFilestat(GitBlitSuite.REPOSITORIES);
         assertNotNull(stat);
         assertTrue(stat.mode > 0);
         assertTrue(stat.uid > 0);
         assertTrue(stat.gid > 0);
      }
   }


}