James Moger
2014-04-17 9ee8b1b793113e74a2fffc1e2aa764279d1c686d
[findbugs] Null checks on closing a repository in ITicketService
1 files modified
32 ■■■■ changed files
src/main/java/com/gitblit/tickets/ITicketService.java 32 ●●●● patch | view | raw | blame | history
src/main/java/com/gitblit/tickets/ITicketService.java
@@ -379,7 +379,9 @@
        } catch (IOException e) {
            log.error("failed to create label " + label + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return lb;
    }
@@ -405,7 +407,9 @@
        } catch (IOException e) {
            log.error("failed to update label " + label + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }
@@ -444,7 +448,9 @@
        } catch (IOException e) {
            log.error("failed to rename label " + oldName + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }
@@ -473,7 +479,9 @@
        } catch (IOException e) {
            log.error("failed to delete label " + label + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }
@@ -579,7 +587,9 @@
        } catch (IOException e) {
            log.error("failed to create milestone " + milestone + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return ms;
    }
@@ -611,7 +621,9 @@
        } catch (IOException e) {
            log.error("failed to update milestone " + milestone + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }
@@ -659,7 +671,9 @@
        } catch (IOException e) {
            log.error("failed to rename milestone " + oldName + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }
@@ -690,7 +704,9 @@
        } catch (IOException e) {
            log.error("failed to delete milestone " + milestone + " in " + repository, e);
        } finally {
            db.close();
            if (db != null) {
                db.close();
            }
        }
        return false;
    }