James Moger
2015-11-18 def01db5f7dc1c607026528eed1dfd75c6e59779
Catch some potential NPEs
2 files modified
10 ■■■■■ changed files
src/main/java/com/gitblit/manager/RepositoryManager.java 7 ●●●●● patch | view | raw | blame | history
src/main/java/com/gitblit/wicket/pages/ForksPage.java 3 ●●●●● patch | view | raw | blame | history
src/main/java/com/gitblit/manager/RepositoryManager.java
@@ -1113,9 +1113,16 @@
            // find the root, cached
            String key = getRepositoryKey(repository);
            RepositoryModel model = repositoryListCache.get(key);
            if (model == null) {
                return null;
            }
            while (model.originRepository != null) {
                String originKey = getRepositoryKey(model.originRepository);
                model = repositoryListCache.get(originKey);
                if (model == null) {
                    return null;
                }
            }
            ForkModel root = getForkModelFromCache(model.name);
            return root;
src/main/java/com/gitblit/wicket/pages/ForksPage.java
@@ -136,6 +136,9 @@
    protected List<FlatFork> flatten(ForkModel node, int level) {
        List<FlatFork> list = new ArrayList<FlatFork>();
        if (node == null) {
            return list;
        }
        list.add(new FlatFork(node.repository, level));
        if (!node.isLeaf()) {
            for (ForkModel fork : node.forks) {