Hi Mladen,

[EMAIL PROTECTED] schrieb:
> @@ -1220,12 +1220,25 @@
...
>          /* Connect to the backend.
>           * This can be either uninitalized connection or a reconnect.
>           */
>          if (ajp_connect_to_endpoint(ae, l) == JK_TRUE) {
> +            /* should we send a CPING to validate connection ? */
> +            if (ae->worker->connect_timeout > 0) {
> +                if (ajp_handle_cping_cpong(ae,
> +                        ae->worker->connect_timeout, l) == JK_FALSE) {
> +                    /* Close the socket if unable to cping/cpong */
> +                    jk_close_socket(ae->sd);
> +                    ae->sd = JK_INVALID_SOCKET;
> +                    JK_TRACE_EXIT(l);
> +                    return JK_FALSE;
> +                }
> +            }
> +            

Cping/Cpong is already done inside ajp_connect_to_endpoint(), if it
fails we take the else part, that does the jk_close_socket. So I think
this would lead to a double cping/cpong after connect.

Regards,

Rainer

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to