Fixed bug #51610 (Using oci_connect causes PHP to take a long time to exit). Do PECL...

diff --git a/ext/oci8/oci8.c b/ext/oci8/oci8.c
index 38a69100f4229205d409add7dba02a5a2d059d20..6fd1389117abf13b281cf374d2b741a0c80ff091 100644 (file)
--- a/ext/oci8/oci8.c
+++ b/ext/oci8/oci8.c
@@ -2694,20 +2694,6 @@ static php_oci_spool *php_oci_create_spool(char *username, int username_len, cha
        if (OCI_G(errcode) != OCI_SUCCESS) {
                php_oci_error(OCI_G(err), OCI_G(errcode) TSRMLS_CC);
                iserror = 1;
-               goto exit_create_spool;
-       }
-
-       /* Set the session pool's timeout to the oci8.persistent_timeout param */
-       if (OCI_G(persistent_timeout)) {
-               ub4 timeout = OCI_G(persistent_timeout);
-
-               PHP_OCI_CALL_RETURN(OCI_G(errcode), OCIAttrSet, ((dvoid *) session_pool->poolh, (ub4) OCI_HTYPE_SPOOL, (void *) &timeout, (ub4) sizeof(timeout), (ub4) OCI_ATTR_SPOOL_TIMEOUT, OCI_G(err)));
-
-               if (OCI_G(errcode) != OCI_SUCCESS) {
-                       php_oci_error(OCI_G(err), OCI_G(errcode) TSRMLS_CC);
-                       iserror = 1;
-                       goto exit_create_spool;
-               }
        }
 
 exit_create_spool:

diff --git a/ext/oci8/package.xml b/ext/oci8/package.xml
index 3e5907b0dd28b0f3d5310ff0374df2f992a75df9..de5c8a04fd8033ac891a128d6a7b3a363d2bfd12 100644 (file)
--- a/ext/oci8/package.xml
+++ b/ext/oci8/package.xml
@@ -33,12 +33,12 @@ http://pear.php.net/dtd/package-2.0.xsd">
   <active>no</active>
  </lead>
 
- <date>2010-06-25</date>
+ <date>2010-08-06</date>
  <time>15:00:00</time>
 
  <version>
-  <release>1.4.2</release>
-  <api>1.4.2</api>
+  <release>1.4.3</release>
+  <api>1.4.3</api>
  </version>
  <stability>
   <release>stable</release>
@@ -46,10 +46,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
  </stability>
  <license uri="http://www.php.net/license">PHP</license>
  <notes>
-    Fixed bug #51691 (Unnecessary realloc causes crashes in PHP trunk tests with interned strings)
-    Fixed bug #51291 (oci_error doesn't report last error when called two times)
-    Fixed bug #51577 (Uninitialized memory reference with oci_bind_array_by_name)
-    Fixed bug #52186 (phpinfo() shows 10.1 or 11.1 when installed with ORACLE_HOME 10.2 or 11.2)
+    Fixed bug #51610 (Using oci_connect causes PHP to take a long time to exit). Requires Oracle bug fix 9891199 for this patch to have an effect.
  </notes>
  <contents>
   <dir name="/">
@@ -309,6 +306,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
     <file name="lob_temp.phpt" role="test" />
     <file name="minfo.phpt" role="test" />
     <file name="num.phpt" role="test" />
+    <file name="oci8safemode.phpt" role="test" />
     <file name="oci_execute_segfault.phpt" role="test" />
     <file name="old_oci_close1.phpt" role="test" />
     <file name="old_oci_close.phpt" role="test" />
@@ -378,6 +376,26 @@ http://pear.php.net/dtd/package-2.0.xsd">
  </extsrcrelease>
  <changelog>
 
+<release>
+ <version>
+  <release>1.4.2</release>
+  <api>1.4.2</api>
+ </version>
+ <stability>
+  <release>stable</release>
+  <api>stable</api>
+ </stability>
+ <license uri="http://www.php.net/license">PHP</license>
+ <notes>
+    Fixed bug #52186 (phpinfo() shows 10.1 or 11.1 when installed with ORACLE_HOME 10.2 or 11.2)
+    Fixed bug #51691 (Unnecessary realloc causes crashes in PHP trunk tests with interned strings)
+    Fixed bug #51577 (Uninitialized memory reference with oci_bind_array_by_name)
+    Fixed bug #51291 (oci_error doesn't report last error when called two times)
+
+    OCI8 1.4.2 is included in PHP 5.3.3
+ </notes>
+</release>
+
 <release>
  <version>
   <release>1.4.1</release>

diff --git a/ext/oci8/php_oci8.h b/ext/oci8/php_oci8.h
index a79702efd336e78accdd3a3db93003e0dd766847..9bf589d660abfc723415bee3384d16029668f7d2 100644 (file)
--- a/ext/oci8/php_oci8.h
+++ b/ext/oci8/php_oci8.h
@@ -46,7 +46,7 @@
  */
 #undef PHP_OCI8_VERSION
 #endif
-#define PHP_OCI8_VERSION "1.4.2"
+#define PHP_OCI8_VERSION "1.4.3"
 
 extern zend_module_entry oci8_module_entry;
 #define phpext_oci8_ptr &oci8_module_entry