ignatenkobrain / rpms / nginx

Forked from rpms/nginx 2 years ago
Clone
6d5d27
diff --git a/src/event/ngx_event_openssl.c b/src/event/ngx_event_openssl.c
6d5d27
index 2b0c5e6..9278087 100644
6d5d27
--- a/src/event/ngx_event_openssl.c
6d5d27
+++ b/src/event/ngx_event_openssl.c
6d5d27
@@ -258,6 +258,8 @@ ngx_ssl_init(ngx_log_t *log)
6d5d27
 ngx_int_t
6d5d27
 ngx_ssl_create(ngx_ssl_t *ssl, ngx_uint_t protocols, void *data)
6d5d27
 {
6d5d27
+    ngx_uint_t prot = NGX_SSL_NO_PROT;
6d5d27
+
6d5d27
     ssl->ctx = SSL_CTX_new(SSLv23_method());
6d5d27
 
6d5d27
     if (ssl->ctx == NULL) {
6d5d27
@@ -322,49 +324,54 @@ ngx_ssl_create(ngx_ssl_t *ssl, ngx_uint_t protocols, void *data)
6d5d27
 
6d5d27
     SSL_CTX_set_options(ssl->ctx, SSL_OP_SINGLE_DH_USE);
6d5d27
 
6d5d27
-#if OPENSSL_VERSION_NUMBER >= 0x009080dfL
6d5d27
-    /* only in 0.9.8m+ */
6d5d27
-    SSL_CTX_clear_options(ssl->ctx,
6d5d27
-                          SSL_OP_NO_SSLv2|SSL_OP_NO_SSLv3|SSL_OP_NO_TLSv1);
6d5d27
-#endif
6d5d27
-
6d5d27
-    if (!(protocols & NGX_SSL_SSLv2)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_SSLv2);
6d5d27
-    }
6d5d27
-    if (!(protocols & NGX_SSL_SSLv3)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_SSLv3);
6d5d27
-    }
6d5d27
-    if (!(protocols & NGX_SSL_TLSv1)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_TLSv1);
6d5d27
-    }
6d5d27
-#ifdef SSL_OP_NO_TLSv1_1
6d5d27
-    SSL_CTX_clear_options(ssl->ctx, SSL_OP_NO_TLSv1_1);
6d5d27
-    if (!(protocols & NGX_SSL_TLSv1_1)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_TLSv1_1);
6d5d27
-    }
6d5d27
+    if (protocols){
6d5d27
+#ifdef SSL_OP_NO_TLSv1_3
6d5d27
+        if (protocols & NGX_SSL_TLSv1_3) {
6d5d27
+            prot = TLS1_3_VERSION;
6d5d27
+        } else
6d5d27
 #endif
6d5d27
 #ifdef SSL_OP_NO_TLSv1_2
6d5d27
-    SSL_CTX_clear_options(ssl->ctx, SSL_OP_NO_TLSv1_2);
6d5d27
-    if (!(protocols & NGX_SSL_TLSv1_2)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_TLSv1_2);
6d5d27
-    }
6d5d27
+        if (protocols & NGX_SSL_TLSv1_2) {
6d5d27
+            prot =  TLS1_2_VERSION;
6d5d27
+        } else
6d5d27
 #endif
6d5d27
-#ifdef SSL_OP_NO_TLSv1_3
6d5d27
-    SSL_CTX_clear_options(ssl->ctx, SSL_OP_NO_TLSv1_3);
6d5d27
-    if (!(protocols & NGX_SSL_TLSv1_3)) {
6d5d27
-        SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_TLSv1_3);
6d5d27
-    }
6d5d27
+#ifdef SSL_OP_NO_TLSv1_1
6d5d27
+        if (protocols & NGX_SSL_TLSv1_1) {
6d5d27
+            prot = TLS1_1_VERSION;
6d5d27
+        } else
6d5d27
 #endif
6d5d27
+        if (protocols & NGX_SSL_TLSv1) {
6d5d27
+            prot = TLS1_VERSION;
6d5d27
+        }
6d5d27
+
6d5d27
+        if (prot == NGX_SSL_NO_PROT) {
6d5d27
+                    ngx_ssl_error(NGX_LOG_EMERG, ssl->log, 0,
6d5d27
+                      "No SSL protocols available [hint: ssl_protocols]");
6d5d27
+            return NGX_ERROR;
6d5d27
+        }
6d5d27
 
6d5d27
-#ifdef SSL_CTX_set_min_proto_version
6d5d27
-    SSL_CTX_set_min_proto_version(ssl->ctx, 0);
6d5d27
-    SSL_CTX_set_max_proto_version(ssl->ctx, TLS1_2_VERSION);
6d5d27
+        SSL_CTX_set_max_proto_version(ssl->ctx, prot);
6d5d27
+
6d5d27
+        /* Now, we have to scan for minimal protocol version,
6d5d27
+         *without allowing holes between min and max*/
6d5d27
+#if SSL_OP_NO_TLSv1_3
6d5d27
+        if ((prot == TLS1_3_VERSION) && (protocols & NGX_SSL_TLSv1_2)) {
6d5d27
+            prot = TLS1_2_VERSION;
6d5d27
+        }
6d5d27
 #endif
6d5d27
 
6d5d27
-#ifdef TLS1_3_VERSION
6d5d27
-    SSL_CTX_set_min_proto_version(ssl->ctx, 0);
6d5d27
-    SSL_CTX_set_max_proto_version(ssl->ctx, TLS1_3_VERSION);
6d5d27
+#ifdef SSL_OP_NO_TLSv1_1
6d5d27
+        if ((prot == TLS1_2_VERSION) && (protocols & NGX_SSL_TLSv1_1)) {
6d5d27
+            prot = TLS1_1_VERSION;
6d5d27
+        }
6d5d27
+#endif
6d5d27
+#ifdef SSL_OP_NO_TLSv1_2
6d5d27
+        if ((prot == TLS1_1_VERSION) && (protocols & NGX_SSL_TLSv1)) {
6d5d27
+            prot = TLS1_VERSION;
6d5d27
+        }
6d5d27
 #endif
6d5d27
+        SSL_CTX_set_min_proto_version(ssl->ctx, prot);
6d5d27
+    }
6d5d27
 
6d5d27
 #ifdef SSL_OP_NO_COMPRESSION
6d5d27
     SSL_CTX_set_options(ssl->ctx, SSL_OP_NO_COMPRESSION);
6d5d27
diff --git a/src/event/ngx_event_openssl.h b/src/event/ngx_event_openssl.h
6d5d27
index 329760d..5cee113 100644
6d5d27
--- a/src/event/ngx_event_openssl.h
6d5d27
+++ b/src/event/ngx_event_openssl.h
6d5d27
@@ -152,6 +152,7 @@ typedef struct {
6d5d27
 #endif
6d5d27
 
6d5d27
 
6d5d27
+#define NGX_SSL_NO_PROT  0x0000
6d5d27
 #define NGX_SSL_SSLv2    0x0002
6d5d27
 #define NGX_SSL_SSLv3    0x0004
6d5d27
 #define NGX_SSL_TLSv1    0x0008
6d5d27
diff --git a/src/http/modules/ngx_http_ssl_module.c b/src/http/modules/ngx_http_ssl_module.c
6d5d27
index a47d696..94f30db 100644
6d5d27
--- a/src/http/modules/ngx_http_ssl_module.c
6d5d27
+++ b/src/http/modules/ngx_http_ssl_module.c
6d5d27
@@ -671,8 +671,7 @@ ngx_http_ssl_merge_srv_conf(ngx_conf_t *cf, void *parent, void *child)
6d5d27
     ngx_conf_merge_value(conf->reject_handshake, prev->reject_handshake, 0);
6d5d27
 
6d5d27
     ngx_conf_merge_bitmask_value(conf->protocols, prev->protocols,
6d5d27
-                         (NGX_CONF_BITMASK_SET|NGX_SSL_TLSv1
6d5d27
-                          |NGX_SSL_TLSv1_1|NGX_SSL_TLSv1_2));
6d5d27
+                         0)
6d5d27
 
6d5d27
     ngx_conf_merge_size_value(conf->buffer_size, prev->buffer_size,
6d5d27
                          NGX_SSL_BUFSIZE);
6d5d27
diff --git a/src/mail/ngx_mail_ssl_module.c b/src/mail/ngx_mail_ssl_module.c
6d5d27
index 7eae83e..8328560 100644
6d5d27
--- a/src/mail/ngx_mail_ssl_module.c
6d5d27
+++ b/src/mail/ngx_mail_ssl_module.c
6d5d27
@@ -306,8 +306,7 @@ ngx_mail_ssl_merge_conf(ngx_conf_t *cf, void *parent, void *child)
6d5d27
                          prev->prefer_server_ciphers, 0);
6d5d27
 
6d5d27
     ngx_conf_merge_bitmask_value(conf->protocols, prev->protocols,
6d5d27
-                         (NGX_CONF_BITMASK_SET|NGX_SSL_TLSv1
6d5d27
-                          |NGX_SSL_TLSv1_1|NGX_SSL_TLSv1_2));
6d5d27
+                         0);
6d5d27
 
6d5d27
     ngx_conf_merge_uint_value(conf->verify, prev->verify, 0);
6d5d27
     ngx_conf_merge_uint_value(conf->verify_depth, prev->verify_depth, 1);
6d5d27
diff --git a/src/stream/ngx_stream_ssl_module.c b/src/stream/ngx_stream_ssl_module.c
6d5d27
index d8c0471..cef590d 100644
6d5d27
--- a/src/stream/ngx_stream_ssl_module.c
6d5d27
+++ b/src/stream/ngx_stream_ssl_module.c
6d5d27
@@ -641,8 +641,7 @@ ngx_stream_ssl_merge_conf(ngx_conf_t *cf, void *parent, void *child)
6d5d27
                          prev->prefer_server_ciphers, 0);
6d5d27
 
6d5d27
     ngx_conf_merge_bitmask_value(conf->protocols, prev->protocols,
6d5d27
-                         (NGX_CONF_BITMASK_SET|NGX_SSL_TLSv1
6d5d27
-                          |NGX_SSL_TLSv1_1|NGX_SSL_TLSv1_2));
6d5d27
+                         0);
6d5d27
 
6d5d27
     ngx_conf_merge_uint_value(conf->verify, prev->verify, 0);
6d5d27
     ngx_conf_merge_uint_value(conf->verify_depth, prev->verify_depth, 1);