Re: [PATCH 4/8] winhttp: Implement WinHttpWebSocketClose.

[Alistair Leslie-Hughes]

Hi Hans,

On 24/6/20 6:33 pm, Hans Leidekker wrote:
> - if ((ret = receive_bytes( netconn, (char *)&socket->status, sizeof(socket->status), &count ))) goto done;
> - if (count != sizeof(socket->status))
> - {
> -
>    ret = ERROR_WINHTTP_INVALID_SERVER_RESPONSE;
> -
>    goto done;
> - }
> - socket->status = RtlUshortByteSwap( socket->status );
> - ret = receive_bytes( netconn, socket->reason, sizeof(socket->reason), &socket->reason_len );
> -

The status and reason message are optional for the close packet and is 
causing an error on my test application.  The packet length will show if 
it has a code.

https://tools.ietf.org/html/rfc6455#section-5.5.1
If there is a body, the first two bytes of
    the body MUST be a 2-byte unsigned integer (in network byte order)
    representing a status code with value /code/ defined in Section 7.4.
    Following the 2-byte integer, the body MAY contain UTF-8-encoded data
    with value /reason/ating a reason for closure, followed by
    UTF-8-encoded data"

Regards
Alistair.