fix PECL bug #9061 (oci8 might reuse wrong persistent connection)

diff --git a/ext/oci8/oci8.c b/ext/oci8/oci8.c
index 8314e71fe1553fea9b6097fcdbced20a25ec1d7c..0956504a32a97ce4ad4162cb66c09271a5c435b5 100644 (file)
--- a/ext/oci8/oci8.c
+++ b/ext/oci8/oci8.c
@@ -1104,15 +1104,19 @@ php_oci_connection *php_oci_do_connect_ex(char *username, int username_len, char
                                                        /* server died */
                                                } else {
                                                        int rsrc_type;
+                                                       php_oci_connection *tmp;
 
                                                        /* okay, the connection is open and the server is still alive */
                                                        connection->used_this_request = 1;
-                                                       smart_str_free_ex(&hashed_details, 0);
-                                                       if (zend_list_find(connection->rsrc_id, &rsrc_type) && (rsrc_type == le_pconnection) && zend_list_addref(connection->rsrc_id) == SUCCESS) {
+                                                       tmp = (php_oci_connection *)zend_list_find(connection->rsrc_id, &rsrc_type);
+
+                                                       if (tmp != NULL && rsrc_type == le_pconnection && strlen(tmp->hash_key) == hashed_details.len &&
+                                                               memcmp(tmp->hash_key, hashed_details.c, hashed_details.len) == 0 && zend_list_addref(connection->rsrc_id) == SUCCESS) {
                                                                /* do nothing */
                                                        } else {
                                                                connection->rsrc_id = zend_list_insert(connection, le_pconnection);
                                                        }
+                                                       smart_str_free_ex(&hashed_details, 0);
                                                        return connection;
                                                }
                                        }