diff options
| author | James Moger <james.moger@gitblit.com> | 2013-07-19 07:47:56 -0700 |
|---|---|---|
| committer | James Moger <james.moger@gitblit.com> | 2013-07-19 07:47:56 -0700 |
| commit | 42e19382d28d06eda0ffbcda01fbfed5c3538dd8 (patch) | |
| tree | 8e27614ea55e565d2a61de87d0da0f2601ab9b45 /src/main/java/com/gitblit/LogoServlet.java | |
| parent | 58708c13b6cfc1e8289d1dca698d6d228509c695 (diff) | |
| parent | fbe265fa787e4be9cb63c6bae2ef30b9400d9afc (diff) | |
| download | gitblit-42e19382d28d06eda0ffbcda01fbfed5c3538dd8.tar.gz gitblit-42e19382d28d06eda0ffbcda01fbfed5c3538dd8.zip | |
Merge pull request #98 from simonharrer/close-resources-correctly
Fixes findbugs warning - dereferencing null in exception case
Diffstat (limited to 'src/main/java/com/gitblit/LogoServlet.java')
| -rw-r--r-- | src/main/java/com/gitblit/LogoServlet.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/main/java/com/gitblit/LogoServlet.java b/src/main/java/com/gitblit/LogoServlet.java index 166bf40f..c8820ed3 100644 --- a/src/main/java/com/gitblit/LogoServlet.java +++ b/src/main/java/com/gitblit/LogoServlet.java @@ -87,7 +87,9 @@ public class LogoServlet extends HttpServlet { } catch (Exception e) {
e.printStackTrace();
} finally {
- is.close();
+ if(is != null) {
+ is.close();
+ }
}
}
}
|