Browse Source

Merge branch 'master' of git.php.net:php-src

* 'master' of git.php.net:php-src:
  fix missing var for phpize
  Add support for OpenSSL security level
pull/2637/head
Dmitry Stogov 9 years ago
parent
commit
ff83560ec6
  1. 2
      NEWS
  2. 48
      ext/openssl/tests/stream_security_level.phpt
  3. 17
      ext/openssl/xp_ssl.c

2
NEWS

@ -59,6 +59,8 @@ PHP NEWS
(Nikita, Dmitry)
- OpenSSL:
. Add ssl security_level stream option to support OpenSSL security levels.
(Jakub Zelenka).
. Allow setting SNI cert and private key in separate files. (Jakub Zelenka)
. Fixed bug #74651 (negative-size-param (-1) in memcpy in zif_openssl_seal()).
(Stas)

48
ext/openssl/tests/stream_security_level.phpt

@ -0,0 +1,48 @@
--TEST--
security_level setting to prohibit cert
--SKIPIF--
<?php
if (!extension_loaded("openssl")) die("skip openssl not loaded");
if (OPENSSL_VERSION_NUMBER < 0x10100000) die("skip OpenSSL >= v1.1.0 required");
if (!function_exists("proc_open")) die("skip no proc_open");
--FILE--
<?php
$serverCode = <<<'CODE'
$serverUri = "ssl://127.0.0.1:64322";
$serverFlags = STREAM_SERVER_BIND | STREAM_SERVER_LISTEN;
$serverCtx = stream_context_create(['ssl' => [
'local_cert' => __DIR__ . '/bug54992.pem'
]]);
$server = stream_socket_server($serverUri, $errno, $errstr, $serverFlags, $serverCtx);
phpt_notify();
@stream_socket_accept($server, 1);
CODE;
$clientCode = <<<'CODE'
$serverUri = "ssl://127.0.0.1:64322";
$clientFlags = STREAM_CLIENT_CONNECT;
$clientCtx = stream_context_create(['ssl' => [
'security_level' => 3,
'verify_peer' => true,
'cafile' => __DIR__ . '/bug54992-ca.pem',
'verify_peer_name' => false
]]);
phpt_wait();
$client = stream_socket_client($serverUri, $errno, $errstr, 1, $clientFlags, $clientCtx);
var_dump($client);
CODE;
include 'ServerClientTestCase.inc';
ServerClientTestCase::getInstance()->run($clientCode, $serverCode);
--EXPECTF--
Warning: stream_socket_client(): SSL operation failed with code 1. OpenSSL Error messages:
error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed in %s : eval()'d code on line %d
Warning: stream_socket_client(): Failed to enable crypto in %s : eval()'d code on line %d
Warning: stream_socket_client(): unable to connect to ssl://127.0.0.1:64322 (Unknown error) in %s : eval()'d code on line %d
bool(false)

17
ext/openssl/xp_ssl.c

@ -70,6 +70,9 @@
#endif
#endif
#if OPENSSL_VERSION_NUMBER >= 0x10100000L && !defined(LIBRESSL_VERSION_NUMBER)
#define HAVE_SEC_LEVEL 1
#endif
/* Flags for determining allowed stream crypto methods */
#define STREAM_CRYPTO_IS_CLIENT (1<<0)
@ -1613,6 +1616,20 @@ int php_openssl_setup_crypto(php_stream *stream,
}
}
if (GET_VER_OPT("security_level")) {
#ifdef HAVE_SEC_LEVEL
convert_to_long(val);
if (Z_LVAL_P(val) < 0 || Z_LVAL_P(val) > 5) {
php_error_docref(NULL, E_WARNING, "Security level must be between 0 and 5");
}
SSL_CTX_set_security_level(sslsock->ctx, Z_LVAL_P(val));
#else
php_error_docref(NULL, E_WARNING,
"security_level is not supported by the linked OpenSSL library "
"- it is supported from version 1.1.0");
#endif
}
GET_VER_OPT_STRING("alpn_protocols", alpn_protocols);
if (alpn_protocols) {
#ifdef HAVE_TLS_ALPN

Loading…
Cancel
Save