Edit report at http://bugs.php.net/bug.php?id=54230&edit=1
ID: 54230
User updated by: bas at baspeters dot com
Reported by: bas at baspeters dot com
Summary: stream_socket_accept leaks memory on SSL protocol
using peer parameter
Status: Open
Type: Bug
Package: Sockets related
-Operating System: Linux, OSX
+Operating System: Linux, FreeBSD, OSX
PHP Version: 5.3.5
Block user comment: N
Private report: N
New Comment:
I have managed to reproduce this behavior on the following systems:
FreeBSD 6.2 - PHP 5.4.2 - OpenSSL 0.9.7e-p1
FreeBSD 6.1 - PHP 5.2.0 - OpenSSL 0.9.7e-p1
Centos 2.6.18-194.32.1.el5 - PHP 5.3.5 - OpenSSL 0.9.8e-fips-rhel5
OSX 10.6.6 - PHP 5.3.5 - OpenSSL 1.0.0d
Memory leak may be accelerated by pressing and holding F5 / CTRL-R /
CMD-R in the browser window causing it to refresh at a rapid pace.
Previous Comments:
------------------------------------------------------------------------
[2011-03-12 00:14:28] felipe@php.net
I can't reproduce it. (OpenSSL 0.9.8g)
int(348020)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
int(348236)
------------------------------------------------------------------------
[2011-03-11 21:38:40] bas at baspeters dot com
For those who want to test but are unable to supply the required
certificate.pem file, here is a link:
http://www.baspeters.com/patches/certificate.pem
password is 'password' as stated in the test script.
I have changed the Package from 'OpenSSL' to 'Sockets related' because I
think it is a better match.
------------------------------------------------------------------------
[2011-03-11 20:40:03] bas at baspeters dot com
Description:
------------
I encountered a memory leak issue when creating an SSL enabled socket
server in a long lived process. When an incoming listening socket is
accepted on either the SSL or TLS protocol, the &$peer passthrough
variable in the stream_socket_accept function causes a memory leak.
NOTE: The test script below needs a valid certificate.pem file to
work!!
NOTE: The test script spawns a webserver to be reached on
127.0.0.1:8888
NOTE: Testing can be done in a browser by choosing appropriate protocol
(http/https) and perform regular page refreshes
Test matrix:
SSL=true, PEER=true -> memory leak!
SSL=false, PEER=true -> no memory leak
SSL=true, PEER=false -> no memory leak
SSL=false, PEER=false -> no memory leak
Test script:
---------------
define('SSL', true);
define('PEER', true);
$context = stream_context_create();
stream_context_set_option($context, 'ssl', 'local_cert',
'certificate.pem');
stream_context_set_option($context, 'ssl', 'passphrase', 'password');
stream_context_set_option($context, 'ssl', 'allow_self_signed', true);
if(SSL) {
$server = stream_socket_server("ssl://0.0.0.0:8888", $errno, $errstr,
STREAM_SERVER_BIND|STREAM_SERVER_LISTEN, $context);
} else {
$server = stream_socket_server("tcp://0.0.0.0:8888", $errno, $errstr,
STREAM_SERVER_BIND|STREAM_SERVER_LISTEN);
}
while(true) {
if(PEER) { $client = @stream_socket_accept($server, 2, $peer); } else
{ $client = @stream_socket_accept($server, 2); }
if($client === false) continue;
@fwrite($client, "HTTP/1.1 200 OK\r\n\r\nMemory usage:
".memory_get_usage().' bytes');
@fclose($client);
}
Expected result:
----------------
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Memory usage: 641032 bytes
Actual result:
--------------
Memory usage: 641368 bytes
Memory usage: 641560 bytes
Memory usage: 641752 bytes
Memory usage: 641944 bytes
Memory usage: 642232 bytes
Memory usage: 642424 bytes
Memory usage: 642616 bytes
Memory usage: 642808 bytes
Memory usage: 643000 bytes
Memory usage: 643192 bytes
Memory usage: 643384 bytes
Memory usage: 643576 bytes
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=54230&edit=1