#38210 [NEW]: oci8.ping_interval interpreted incorrectly

From: Paul dot Merchant at dartmouth dot edu Operating system: Any PHP version: 5.1.4 PHP Bug Type: OCI8 related Bug description: oci8.ping_interval interpreted incorrectly Description: ------------ A ping interval of 0 should cause a persistent connection to be polled each time php_oci_do_connect is called. It does this by adding ping_interval seconds to the timestamp of the last operation to compute a next ping time; however, php_oci_do_connect only performs a ping when the current timestamp is greater than the next ping time, guaranteeing that at least 1 second must pass before the next ping occurs. Effectively, php_oci_connection_ping is called every ping_interval+1 seconds. The comparison on php_oci_do_connect should read if ( (connection->next_ping > 0) && (timestamp >= connection- >next_ping) &&.... -- Edit bug report at http://bugs.php.net/?id=38210&edit=1 -- Try a CVS snapshot (PHP 4.4): http://bugs.php.net/fix.php?id=38210&r=trysnapshot44 Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=38210&r=trysnapshot52 Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=38210&r=trysnapshot60 Fixed in CVS: http://bugs.php.net/fix.php?id=38210&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=38210&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=38210&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=38210&r=needscript Try newer version: http://bugs.php.net/fix.php?id=38210&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=38210&r=support Expected behavior: http://bugs.php.net/fix.php?id=38210&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=38210&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=38210&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=38210&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=38210&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=38210&r=dst IIS Stability: http://bugs.php.net/fix.php?id=38210&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=38210&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=38210&r=float No Zend Extensions: http://bugs.php.net/fix.php?id=38210&r=nozend MySQL Configuration Error: http://bugs.php.net/fix.php?id=38210&r=mysqlcfg