From 1fb5424b5532ff58ac6c55510d87d5674cc5d89b Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 17 Apr 2014 23:08:07 -0400
Subject: [PATCH] [findbugs] Null check on closing PAM provider

---
 src/main/java/com/gitblit/auth/PAMAuthProvider.java |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/src/main/java/com/gitblit/auth/PAMAuthProvider.java b/src/main/java/com/gitblit/auth/PAMAuthProvider.java
index 7e82bc6..5d441b8 100644
--- a/src/main/java/com/gitblit/auth/PAMAuthProvider.java
+++ b/src/main/java/com/gitblit/auth/PAMAuthProvider.java
@@ -98,7 +98,9 @@
 			logger.error(e.getMessage());
 			return null;
 		} finally {
-			pam.dispose();
+			if (pam != null) {
+				pam.dispose();
+			}
 		}
 
         UserModel user = userManager.getUserModel(username);

--
Gitblit v1.9.1