php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #19155 fbsql_free_result() must be used
Submitted: 2002-08-28 11:42 UTC Modified: 2002-09-21 02:07 UTC
From: daggillies at yahoo dot com Assigned:
Status: Closed Package: FrontBase related
PHP Version: 4.2.2 OS: Mandrake Linux/FreeBSD
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: daggillies at yahoo dot com
New email:
PHP Version: OS:

 

 [2002-08-28 11:42 UTC] daggillies at yahoo dot com
fbsql_free_result().

This function appears to be of critical importance. If
you are experiencing random hangs and crashes in your scripts, suspect a lack of fbsql_free_result().
It appears that failure to explicitly release a result set causes memory problems, usually manifested by Apache segfaulting. With FrontBase < 3.4, this caused us very serious problems as locks were not being released, resulting in a stalled database. With 3.4/3.5, this no longer happens, but nonetheless the httpd child dies, which means that pages are only partially output, and also impacts on performance. I don't know the internals of the fbsql PHP driver and don't have time to investigate further. I would suspect a bug in how cleanup in the garbage collector is implemented, possibly leading to attempts to free the same memory block twice, or an attempt to free a block with a bad pointer (although that would be more likely to yield a SIGBUS, at least occasionally, and we have only seen SIGSEGV). Simply using fbsql_free_result() without knowing exactly what is causing the problem might look like voodoo, but we have found it to be voodoo that works.

A typical FrontBase session might look like this:

$fbdc=fbsql_connect($DBServer,$DBUser,$DBPass) or die("Could not connect to db"); 
$cok=fbsql_select_db($DBName);
if(!$cok)
 {
 echo "<P>DB select failed</P>\n";
 die();
 }

$qstr="select * from products where qty_on_hand < 25;";

$oresult=fbsql_query($qstr,$fbdc);

if($oresult===false)
 {
 echo "<P>Query [$qstr] failed: ".fbsql_errno().": ".fbsql_error()."</P>\n";
 die();
 }

do
 {
 $orow=fbsql_fetch_row($oresult);
 if($orow!==false)
  {
  ...
  ...
  }
 }while($orow!==false);

fbsql_free_result($oresult); <--- THIS IS CRITICAL

fbsql_close($fbdc);

It is a big job to go back through a large set of scripts and retrofit this code. Although the manual states it should not be necessary, it would appear prudent to use this call in new code.

David Gillies
San Jose
Costa Rica

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2002-09-03 23:25 UTC] kalowsky@php.net
Please try using this CVS snapshot:

  http://snaps.php.net/php4-latest.tar.gz
 
For Windows:
 
  http://snaps.php.net/win32/php4-win32-latest.zip

Can you please try a snapshot dated sometime after today?  I've put in a fix which I believe will fix this.
 [2002-09-21 02:07 UTC] sniper@php.net
No feedback, and should be fixed.

 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Mon Dec 09 01:01:27 2024 UTC