AbstractDriverBasedDataSource does not rely on Properties chaining an…

…ymore Issue: SPR-9461
Hyacinth · Jan 18, 2013 · 5d9ad5b · 5d9ad5b
1 parent 87968e5
commit 5d9ad5b
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/...jdbc/src/main/java/org/springframework/jdbc/datasource/AbstractDriverBasedDataSource.java b/...jdbc/src/main/java/org/springframework/jdbc/datasource/AbstractDriverBasedDataSource.java
@@ -139,15 +139,18 @@ public Connection getConnection(String username, String password) throws SQLExce
 	 * @see java.sql.Driver#connect(String, java.util.Properties)
 	 */
 	protected Connection getConnectionFromDriver(String username, String password) throws SQLException {
-		Properties props = new Properties();
-		props.putAll(getConnectionProperties());
+		Properties mergedProps = new Properties();
+		Properties connProps = getConnectionProperties();
+		if (connProps != null) {
+			mergedProps.putAll(connProps);
+		}
 		if (username != null) {
-			props.setProperty("user", username);
+			mergedProps.setProperty("user", username);
 		}
 		if (password != null) {
-			props.setProperty("password", password);
+			mergedProps.setProperty("password", password);
 		}
-		return getConnectionFromDriver(props);
+		return getConnectionFromDriver(mergedProps);
 	}
 
 	/**