php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

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
View Add Comment Developer Edit
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-2023 The PHP Group
All rights reserved. 		Last updated: Sat Jun 10 04:03:38 2023 UTC