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