php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #38210 oci8.ping_interval interpreted incorrectly
Submitted: 2006-07-25 17:46 UTC Modified: 2006-07-26 07:01 UTC
From: Paul dot Merchant at dartmouth dot edu Assigned:
Status: Closed Package: OCI8 related
PHP Version: 5.1.4 OS: Any
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: Paul dot Merchant at dartmouth dot edu
New email:
PHP Version: OS:

 

 [2006-07-25 17:46 UTC] Paul dot Merchant at dartmouth dot edu
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) &&....


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2006-07-26 07:01 UTC] tony2001@php.net
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.


 
PHP Copyright © 2001-2022 The PHP Group
All rights reserved.
Last updated: Thu Aug 18 19:05:43 2022 UTC