Merge lp:~gaul/mariadb-java-client/callablestatement-leak into lp:mariadb-java-client

Proposed by Andrew Gaul
Status: Needs review
Proposed branch: lp:~gaul/mariadb-java-client/callablestatement-leak
Merge into: lp:mariadb-java-client
Diff against target: 12 lines (+1/-1)
1 file modified
src/main/java/org/mariadb/jdbc/MySQLCallableStatement.java (+1/-1)
To merge this branch: bzr merge lp:~gaul/mariadb-java-client/callablestatement-leak
Reviewer Review Type Date Requested Status
Maria-captains Pending
Review via email: mp+198341@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

499. By Andrew Gaul <email address hidden>

Plug CallableStatement resource leak

Found with FindBugs.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/main/java/org/mariadb/jdbc/MySQLCallableStatement.java'
--- src/main/java/org/mariadb/jdbc/MySQLCallableStatement.java 2013-07-24 12:23:19 +0000
+++ src/main/java/org/mariadb/jdbc/MySQLCallableStatement.java 2013-12-10 07:07:01 +0000
@@ -1409,7 +1409,7 @@
1409 rsOutputParameters = null;1409 rsOutputParameters = null;
1410 }1410 }
14111411
1412 if (callStatement == null) {1412 if (callStatement != null) {
1413 callStatement.close();1413 callStatement.close();
1414 callStatement = null;1414 callStatement = null;
1415 }1415 }

Subscribers

People subscribed via source and target branches