Fix possible NPEs
authorSebastian Bazley <sebb@apache.org>
Sat, 9 Jan 2010 14:10:43 +0000 (14:10 +0000)
committerSebastian Bazley <sebb@apache.org>
Sat, 9 Jan 2010 14:10:43 +0000 (14:10 +0000)
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/dbcp/trunk@897457 13f79535-47bb-0310-9956-ffa450edef68

doc/BasicDataSourceExample.java
doc/JOCLPoolingDriverExample.java
doc/ManualPoolingDataSourceExample.java
doc/ManualPoolingDriverExample.java

index da45c7e..d0b9ad1 100644 (file)
@@ -104,9 +104,9 @@ public class BasicDataSourceExample {
         } catch(SQLException e) {
             e.printStackTrace();
         } finally {
-            try { rset.close(); } catch(Exception e) { }
-            try { stmt.close(); } catch(Exception e) { }
-            try { conn.close(); } catch(Exception e) { }
+            try { if (rset != null) rset.close(); } catch(Exception e) { }
+            try { if (stmt != null) stmt.close(); } catch(Exception e) { }
+            try { if (conn != null) conn.close(); } catch(Exception e) { }
         }
     }
 
index 459dd72..f1691cf 100644 (file)
@@ -111,9 +111,9 @@ public class JOCLPoolingDriverExample {
         } catch(SQLException e) {
             e.printStackTrace();
         } finally {
-            try { rset.close(); } catch(Exception e) { }
-            try { stmt.close(); } catch(Exception e) { }
-            try { conn.close(); } catch(Exception e) { }
+            try { if (rset != null) rset.close(); } catch(Exception e) { }
+            try { if (stmt != null) stmt.close(); } catch(Exception e) { }
+            try { if (conn != null) conn.close(); } catch(Exception e) { }
         }
     }
 }
index 9de63e5..f45af2b 100644 (file)
@@ -127,9 +127,9 @@ public class ManualPoolingDataSourceExample {
         } catch(SQLException e) {
             e.printStackTrace();
         } finally {
-            try { rset.close(); } catch(Exception e) { }
-            try { stmt.close(); } catch(Exception e) { }
-            try { conn.close(); } catch(Exception e) { }
+            try { if (rset != null) rset.close(); } catch(Exception e) { }
+            try { if (stmt != null) stmt.close(); } catch(Exception e) { }
+            try { if (conn != null) conn.close(); } catch(Exception e) { }
         }
     }
 
index 10add3e..f758d1c 100644 (file)
@@ -129,9 +129,9 @@ public class ManualPoolingDriverExample {
         } catch(SQLException e) {
             e.printStackTrace();
         } finally {
-            try { rset.close(); } catch(Exception e) { }
-            try { stmt.close(); } catch(Exception e) { }
-            try { conn.close(); } catch(Exception e) { }
+            try { if (rset != null) rset.close(); } catch(Exception e) { }
+            try { if (stmt != null) stmt.close(); } catch(Exception e) { }
+            try { if (conn != null) conn.close(); } catch(Exception e) { }
         }
 
         // Display some pool statistics