All files from this thread

File Name	File Size		Posted by	Date
0001-Retain-CAP_NET_BIND_SERVICE-capability-for-udp-privi.patch	3.5 KB	open | download	shankerwangmiao	02/21/2018	Read message

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?
_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx