DTLS patches

Vladimir Homutov vl at nginx.com
Wed Feb 21 10:18:58 UTC 2018


On Wed, Feb 21, 2018 at 12:18:27PM +0800, Wang Shanker wrote:
> Hi, Vladimir
>
> `ngx_stream_ssl_init_connection` trys to set tcp_nodelay on the given connection.
> The following patch adds a test for the type of connection before set.
>
> Cheers,
>
> Miao Wang
>
> diff --git a/src/stream/ngx_stream_ssl_module.c b/src/stream/ngx_stream_ssl_module.c
> index f85bbb6..36f7fdd 100644
> --- a/src/stream/ngx_stream_ssl_module.c
> +++ b/src/stream/ngx_stream_ssl_module.c
> @@ -369,7 +369,7 @@ ngx_stream_ssl_init_connection(ngx_ssl_t *ssl, ngx_connection_t *c)
>
>     cscf = ngx_stream_get_module_srv_conf(s, ngx_stream_core_module);
>
> -    if (cscf->tcp_nodelay && ngx_tcp_nodelay(c) != NGX_OK) {
> +    if (cscf->tcp_nodelay && c->type == SOCK_STREAM && ngx_tcp_nodelay(c) != NGX_OK) {
>         return NGX_ERROR;
>     }
>
>

Hi, Miao

The change is indeed correct, it is required since
http://hg.nginx.org/nginx/rev/29c6d66b83ba

Have you tried patches in work?


More information about the nginx mailing list