diff options
| author | Chris Wright <daverandom@php.net> | 2014-08-27 16:06:28 +0100 |
|---|---|---|
| committer | Chris Wright <daverandom@php.net> | 2014-08-27 16:06:28 +0100 |
| commit | db03216e62180000d7296c0498f7141ce62f66d8 (patch) | |
| tree | 3ebc30774bc816a1fa7de8b5aec1d7411ab917f6 /ext | |
| parent | 4d667eeaf39853c4c7cbae9f5eaad40c704bffe8 (diff) | |
| parent | 30a73658c63a91c413305a4c4d49882fda4dab3e (diff) | |
| download | php-git-db03216e62180000d7296c0498f7141ce62f66d8.tar.gz | |
Merge branch 'PHP-5.5' into PHP-5.6
* PHP-5.5:
Fix stream_select() issue with OpenSSL buffer
Diffstat (limited to 'ext')
| -rw-r--r-- | ext/openssl/xp_ssl.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/ext/openssl/xp_ssl.c b/ext/openssl/xp_ssl.c index 0a85d77aaa..1af8b12115 100644 --- a/ext/openssl/xp_ssl.c +++ b/ext/openssl/xp_ssl.c @@ -2176,6 +2176,19 @@ static int php_openssl_sockop_cast(php_stream *stream, int castas, void **ret TS case PHP_STREAM_AS_FD_FOR_SELECT: if (ret) { + if (sslsock->ssl_active) { + /* OpenSSL has an internal buffer which select() cannot see. If we don't + fetch it into the stream's buffer, no activity will be reported on the + stream even though there is data waiting to be read - but we only fetch + the number of bytes OpenSSL has ready to give us since we weren't asked + for any data at this stage. This is only likely to cause issues with + non-blocking streams, but it's harmless to always do it. */ + int bytes; + while ((bytes = SSL_pending(sslsock->ssl_handle)) > 0) { + php_stream_fill_read_buffer(stream, (size_t)bytes); + } + } + *(php_socket_t *)ret = sslsock->s.socket; } return SUCCESS; |
