Re: [PATCH] wininet: Initialize netconn_stream's content_length

[Andy Clayton]

On Tue, Nov 22, 2011 at 9:28 AM, Andy Clayton  wrote:
> Hi Jacek,
>
> On Tue, Nov 22, 2011 at 3:59 AM, Jacek Caban wrote:
> > 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.
>
> Thanks for looking over my patch. So it seems then that
> HTTP_HttpSendRequestW should not be calling HTTP_ReceiveRequestData if
> bEndRequest is false (http.c:4941), since there is no way at that point it
> could have read the headers. Is that a better fix?
>
> Thanks again for your help,
>
> Andy

I've attached a second attempt showing this. Thanks for any thoughts you
can provide.

Andy