From 59593af9a3d2f8ac30ecac084cdd1168f8ed2a9d Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Mon, 30 Dec 2013 13:52:26 -0500 Subject: [PATCH] Fix potential NPE on getting Markdown content --- src/main/java/com/gitblit/wicket/pages/SummaryPage.java | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/pages/SummaryPage.java b/src/main/java/com/gitblit/wicket/pages/SummaryPage.java index 9129302..fdc5d01 100644 --- a/src/main/java/com/gitblit/wicket/pages/SummaryPage.java +++ b/src/main/java/com/gitblit/wicket/pages/SummaryPage.java @@ -139,9 +139,12 @@ if (app().settings().getBoolean(Keys.web.summaryShowReadme, false)) { // show a readme on the summary page + MarkupDocument markupDoc = null; RevCommit head = JGitUtils.getCommit(r, null); - MarkupProcessor processor = new MarkupProcessor(app().settings()); - MarkupDocument markupDoc = processor.getReadme(r, repositoryName, getBestCommitId(head)); + if (head != null) { + MarkupProcessor processor = new MarkupProcessor(app().settings()); + markupDoc = processor.getReadme(r, repositoryName, getBestCommitId(head)); + } if (markupDoc == null || markupDoc.markup == null) { add(new Label("readme").setVisible(false)); } else { -- Gitblit v1.9.1