On Wed, Mar 18, 2015 at 10:44 AM, Yann Ylavic <[email protected]> wrote:
>
[]
> Index: modules/proxy/mod_proxy_wstunnel.c
> ===================================================================
> --- modules/proxy/mod_proxy_wstunnel.c (revision 1665828)
> +++ modules/proxy/mod_proxy_wstunnel.c (working copy)
[]
> @@ -292,7 +299,15 @@ static int proxy_wstunnel_request(apr_pool_t *p, r
> ap_log_rerror(APLOG_MARK, APLOG_TRACE2, 0, r,
> "finished with poll() - cleaning up");
>
> - return OK;
> + if (!response_sent) {
> + return HTTP_BAD_GATEWAY;
> + }
> + else if (!request_sent) {
> + return HTTP_BAD_REQUEST;
This case is probably not a good idea, so we'd better not handle
request_sent at all (only response_sent), and use NULL instead for
proxy_wstunnel_transfer().
> + }
> + else {
> + return OK;
> + }
> }
>
> /*
> --
