[PATCH] wininet: Initialize netconn_stream's content_length
Jacek Caban
jacek at codeweavers.com
Tue Nov 22 03:59:55 CST 2011
Hi Andy,
On 11/22/11 5:49 AM, Andy Clayton wrote:
> The size that net_conn_read reads is limited by netconn_stream's
> remaining content_length, so if the content_length is zero it reads
> nothing. It also means that the read size it returns is 0, which
> typically indicates that the socket was closed. As a result of the
> change from 193da884ad9b982d9b740b15453aa8d3c19b054f,
> HTTP_ReceiveRequestData acts on the read size of 0 and closes the
> connection. HttpSendRequest calls HTTP_ReceiveRequestData if there is no
> post data, closing the connection with none of the response data ever
> being read.
>
> This patch fixes this case by initializing netconn_stream's
> content_length to ~0.
>
> Fixes http://bugs.winehq.org/show_bug.cgi?id=27816
>
This doesn't seem right. At the point where we call
HTTP_ReceiveRequestData, content_length is supposed to be set based on
response header. The bug must be somewhere else.
Jacek
More information about the wine-devel
mailing list