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

go to bug id or search bugs for

Bug #81113 PDO holds some non-GCable reference
Submitted: 2021-06-08 08:42 UTC Modified: 2021-06-09 10:06 UTC
From: m at m dot cz Assigned:
Status: Not a bug Package: PDO related
PHP Version: 8.0.7 OS: any
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: m at m dot cz
New email:
PHP Version: OS:

 

 [2021-06-08 08:42 UTC] m at m dot cz 
Description:
------------
See https://github.com/atk4/data/pull/878/commits/83345e3ce4b9006efe9bfe50befacef20f3aee08

When DBAL connection is closed manually - all references to PDO connection are removed manually (by https://github.com/doctrine/dbal/blob/3.1.0/src/Connection.php#L646 ) - then PDO connection is released correctly.

But when there is no reference to an object holding the PDO connection (in this case Doctrine\DBAL\Connection), PDO connection is NOT released when GC is run.

I belive PDO holds some reference which is not decreased during GC cycle.

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2021-06-08 08:46 UTC] m at m dot cz 
working link https://github.com/atk4/data/commit/83345e3ce4b9006efe9bfe50befacef20f3aee08
 [2021-06-08 08:52 UTC] nikic@php.net 
Which PDO driver do you use?
 [2021-06-08 09:31 UTC] m at m dot cz 
At least pdo_mysql is affected, as when max. connection count is limited (by ALTER USER user@% WITH MAX_USER_CONNECTIONS 10;), connection limit is hit.
 [2021-06-08 11:52 UTC] nikic@php.net 
Okay, I think this will need some more reduced code to make progress. PDO generally does support GC, and since PHP 8.1 also supports per-driver GC. So this probably requires that some specific feature is being used.
 [2021-06-08 12:31 UTC] m at m dot cz 
It seems it is related with:
https://github.com/doctrine/dbal/issues/3047
and
https://bugs.php.net/bug.php?id=75190
which describes a GC issue when PDO class is extended, hovewer, it is marked as "not a bug". Why it is not a bug or is it technically impossible to provide full GC support?
 [2021-06-08 12:44 UTC] nikic@php.net 
I tried this script https://gist.github.com/nikic/a55b70d6b7c1acf6785c7e50c37754dc based on the dbal issue, but it does not show a memory leak on PHP 8.1 (or PHP 8.0, or PHP 7.4). So whatever issue that hit, it seems to have been fixed. The memory leak reported in bug #75190 is also fixed.

If there is some reasonably minimal way to reproduce, I'm happy to address other GC problems as well. I don't want to reduce this down from "using doctrine dbal, somewhere, somehow" though.
 [2021-06-09 10:04 UTC] m at m dot cz 
I have futher looked into this issue with following:

a) the main issue is that PHPUnit TestCase is instanciated per test (correct), but PHPUnit does not destroy these instances after the test is completed, but keep that instances referenced until all tests are done (wrong, but PHP works correctly). PHPUnit issue: https://github.com/sebastianbergmann/phpunit/issues/4705

b) https://github.com/atk4/data/commit/0c152f228d27a4ff81223387d881cd7324fe0fb4#diff-2396b34d8258dc49e2597630cd485205d02402dbbc6dd7704c8493cbf9a6ea04R42 even then gc_collect_cycles() is needed to be called at least twice. Reported as https://bugs.php.net/bug.php?id=81117.

This bug report can be closed.
 [2021-06-09 10:06 UTC] nikic@php.net
-Status: Open +Status: Not a bug
 [2021-06-09 10:06 UTC] nikic@php.net 
Closing per above comment.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Dec 26 18:01:31 2024 UTC