diff options
author | Nikita Popov <nikita.ppv@gmail.com> | 2019-06-27 16:52:19 +0200 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2019-06-28 09:18:32 +0200 |
commit | f365d0e00ed93b1c33e984ff3b4cc8677cbca193 (patch) | |
tree | 73577dda0d13024d2104a2bb0f3859857ccca2ae /sapi/phpdbg/phpdbg_io.h | |
parent | a78adce5cb766a3e13328bd0d79d6ac599f3f8bc (diff) | |
download | php-git-f365d0e00ed93b1c33e984ff3b4cc8677cbca193.tar.gz |
Fix mysqlnd memory leak
The actual leak is observed in ext/pdo_mysql/tests/bug_74376.phpt.
The persistent connection leaks because a refcount decrement on a
result is missed. The refcount decrement is missed because
free_result_contents is used, rather than free_result.
Looking at other uses of free_result_contents, it looks like they
could also suffer from this problem. Apart from one case,
free_result_contents is always used to release the result entirely
(I've adjusted the one differing case to only free meta), so I'm
moving most of the logic from free_result into free_result_contents.
The only difference is now that free_result will skip_result first.
Diffstat (limited to 'sapi/phpdbg/phpdbg_io.h')
0 files changed, 0 insertions, 0 deletions