Fix potential NPE in setting shared permissions

diff --git a/src/main/java/com/gitblit/GitBlit.java b/src/main/java/com/gitblit/GitBlit.java
index 95da669e119e67fa6549404bad9934b2f63bc4a6..2cebe82b0f624f0feb6e69c124e30b5174a9b50f 100644 (file)
--- a/src/main/java/com/gitblit/GitBlit.java
+++ b/src/main/java/com/gitblit/GitBlit.java
@@ -2534,16 +2534,16 @@ public class GitBlit implements ServletContextListener {
                                }
                        }
 
+                       // Adjust permissions in case we updated the config files
+                       JGitUtils.adjustSharedPerm(new File(r.getDirectory().getAbsolutePath(), "config"),
+                                       getString(Keys.git.createRepositoriesShared, "FALSE"));
+                       JGitUtils.adjustSharedPerm(new File(r.getDirectory().getAbsolutePath(), "HEAD"),
+                                       getString(Keys.git.createRepositoriesShared, "FALSE"));
+
                        // close the repository object
                        r.close();
                }
 
-               // Adjust permissions in case we updated the config files
-               JGitUtils.adjustSharedPerm(new File(r.getDirectory().getAbsolutePath(), "config"),
-                               getString(Keys.git.createRepositoriesShared, "FALSE"));
-               JGitUtils.adjustSharedPerm(new File(r.getDirectory().getAbsolutePath(), "HEAD"),
-                               getString(Keys.git.createRepositoriesShared, "FALSE"));
-
                // update repository cache
                removeFromCachedRepositoryList(repositoryName);
                // model will actually be replaced on next load because config is stale