Child process (mod_proxy_ws_tunnel worker) go in infinite loop / 100% CPU core usage when browser websocket connection is baldly closed.

mod_proxy_ws_tunnel is configured to proxy websockets:

browser <----conn_client----> apache/mod_proxy_ws_tunnel <----conn_server----> app

If browser or app close properly the connection, there is no problem (worker stop).

If app is killed or conn_server is lost, mod_proxy_ws_tunnel worker stop as expected.

If browser is killed or conn_client is lost, mod_proxy_ws_tunnel worker enter in infinite loop, 100% CPU core usage, and gigabytes log generation ([proxy_wstunnel:debug] AH02445: woke from poll(), i=1).


mod_proxy_wstunnel.c
Method proxy_wstunnel_pump (ap_proxy_wstunnel_request for 2.4.x)


    while(1) { 
        if ((rv = apr_pollset_poll(pollset, timeout, &pollcnt, &signalled))
                != APR_SUCCESS) {
            if (APR_STATUS_IS_EINTR(rv)) {
                continue;
            }
            else if (APR_STATUS_IS_TIMEUP(rv)) { 
                ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(02542) "Attempting to go asynch");
                return SUSPENDED;
            }
            else { 
                ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(02444) "error apr_poll()");
                return HTTP_INTERNAL_SERVER_ERROR;
            }
        }

        ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(02445)
                "woke from poll(), i=%d", pollcnt);

        for (pi = 0; pi < pollcnt; pi++) {
            const apr_pollfd_t *cur = &signalled[pi];

            if (cur->desc.s == sock) {
                pollevent = cur->rtnevents;
                if (pollevent & APR_POLLIN) {
                    ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(02446)
                            "sock was readable");
                    rv = proxy_wstunnel_transfer(r, backconn, c, bb, "sock");
                }
                else if ((pollevent & APR_POLLERR)
                        || (pollevent & APR_POLLHUP)) {
                    rv = APR_EPIPE;
                    ap_log_rerror(APLOG_MARK, APLOG_NOTICE, 0, r, APLOGNO(02447)
                            "err/hup on backconn");
                }
                if (rv != APR_SUCCESS)
                    client_error = 1;
            }
            else if (cur->desc.s == client_socket) {
                pollevent = cur->rtnevents;
                if (pollevent & APR_POLLIN) {
                    ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(02448)
                            "client was readable");
                    rv = proxy_wstunnel_transfer(r, c, backconn, bb, "client");
                }
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
There is no "else if ((pollevent & APR_POLLERR) || (pollevent & APR_POLLHUP))"
Shouldn't be there one?
Copied-pasted above "else if", compiled, tested and worked fine.
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
            }
            else {
                rv = APR_EBADF;
                ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r, APLOGNO(02449)
                        "unknown socket in pollset");
            }

        }
        if (rv != APR_SUCCESS) {
            break;
        }

    }