Re: [PATCH 0/7] MR195: winhttp: Abort web socket receive operations when the socket handle is closed

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/winhttp/request.c:
>  return !ret;
>
> }
>
> +static void socket_handle_closing( struct object_header *hdr )
> +{
> - struct socket *socket = (struct socket *)hdr;
> -
> - if (socket->request->netconn->secure)
> - {
> -
>    cancel_queue( &socket->send_q );
> -
>    cancel_queue( &socket->recv_q );
> - }
> - if (!is_queue_empty( &socket->send_q ) || !is_queue_empty( &socket->recv_q ))
> -
>    netconn_cancel_io( socket->request->netconn );
>
> +}

I doesn't make a lot of sense that async behavior depends on the connection being secure. Do we really need to replicate this?